-
Notifications
You must be signed in to change notification settings - Fork 113
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ignore status for chicklets #3254
ignore status for chicklets #3254
Conversation
Signed-off-by: Rick Marry <[email protected]>
0e2d3b3
to
485b761
Compare
Signed-off-by: Rick Marry <[email protected]>
i'm noticing some differences between the results for nodes and profiles:
note that on the nodes call, we return the correct counts for individual statuses, but the total is affected by the status filter. |
So you agree with the way profiles call responds and are pointing out the the total on the nodes call is off.. correct? |
yes! |
Signed-off-by: Rick Marry <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice thank you!
🔩 Description: What code changed, and why?
When we send it status as a filter for nodes or profiles lists, do not apply the status filter to the actual summary counts. If we apply the status filter to the counts then we won't be able to subsequently choose them in the UI
⛓️ Related Resources
👍 Definition of Done
👟 How to Build and Test the Change
✅ Checklist
📷 Screenshots, if applicable