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
Jamie/3982 data lifecycle additions #4010
Conversation
Signed-off-by: seajamied <jdegnan@chef.io>
Signed-off-by: seajamied <jdegnan@chef.io>
Signed-off-by: seajamied <jdegnan@chef.io>
Signed-off-by: seajamied <jdegnan@chef.io>
Signed-off-by: seajamied <jdegnan@chef.io>
Signed-off-by: seajamied <jdegnan@chef.io>
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.
tooltip for 2nd checkbox is still there, pls remove.. everything else works great, thank you!!
Signed-off-by: seajamied <jdegnan@chef.io>
updated - thank you! |
oh i need to add the docs commit, ill do that tonight |
Signed-off-by: susanev <susan.ra.evans@gmail.com>
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.
π lgtm
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.
Works for me locally. π
// Service Groups are not currently uncheckable through the UI | ||
.filter( form => !['serviceGroupNoHealthChecks', 'serviceGroupRemoveServices'] | ||
.includes(form)), | ||
using(ALL_FORMS.map( form => form ), |
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.
What are you trying to achieve with this no-op map call?
I suspect the filter(identity)
pattern might be more appropriate.
π© Description: What code changed, and why?
This is an update to data-lifecycle that now allows a user to make changes to Service Groups through the UI.
βοΈ Related Resources
API work: #3931
π Definition of Done
A user is now able to successfully change and retain Service groups updates through the UI
π How to Build and Test the Change
build components/automate-ui-devproxy && start_all_services
make serve
navigate to https://a2-dev.test/settings/data-lifecycle
The Service Groups section should no longer be disabled.
Check/UnCheck the Service groups and click the submit button. You should see a success ribbon.
Change the Values in the Service groups section and submit. Check for success ribbon.
Refresh the page - you should see the same values that were previously submitted in the service groups section.
All other sections on the page should retain the same functionality.
β Checklist
π· Screenshots, if applicable
Before:
After: