-
Notifications
You must be signed in to change notification settings - Fork 111
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
[Infra Views Read/Write] Delete Infra Role UI #4792
Conversation
Deploy preview for chef-automate processing. Building with commit 4617f74 https://app.netlify.com/sites/chef-automate/deploys/6041e01fd5d6f10007de0ccc |
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.
Thanks @vinay033 . Looks good to me and works as expected.
Signed-off-by: Vinay Sharma <vsharma@chef.io>
Signed-off-by: Vinay Sharma <vsharma@chef.io>
Signed-off-by: Vinay Sharma <vsharma@chef.io>
Signed-off-by: Vinay Sharma <vsharma@chef.io>
Signed-off-by: Vinay Sharma <vsharma@chef.io>
Signed-off-by: Vinay Sharma <vsharma@chef.io>
Signed-off-by: Vinay Sharma <vsharma@chef.io>
Signed-off-by: Vinay Sharma <vsharma@chef.io>
Signed-off-by: vinay sharma <vsharma@chef.io>
Signed-off-by: Vinay Sharma <vsharma@chef.io>
fe3dfc4
to
9323517
Compare
Signed-off-by: Vinay Sharma <vsharma@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.
The UI looks good.
π© Description: What code changed, and why?
We need to add the UI for the Infra Role delete (Using new UI )
βοΈ Related Resources
#3777
π Definition of Done
I have added a new UI delete modal popup and some other changes to delete the Infra Role.
π How to Build and Test the Change
β Checklist
π· Screenshots, if applicable
delete-role.mp4