Skip to content
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 Client UI #4791

Merged
merged 3 commits into from
Mar 5, 2021
Merged

Conversation

vinay033
Copy link
Collaborator

@vinay033 vinay033 commented Mar 3, 2021

πŸ”© Description: What code changed, and why?

We need to add the UI for the Client delete (Using new UI )

Manage _ Automat (1)

⛓️ Related Resources

#4567

πŸ‘ Definition of Done

I have added a new UI delete modal popup and some other changes to delete the Client.

πŸ‘Ÿ How to Build and Test the Change

Go to the `infrastructure tab` from the top navigation bar then you can see the `Chef Server`
If you have data then you can see the list of servers --> click  to any server ---> you can see the list of orgs --> 
click to any org ---> you can see the multiple tabs after clicking on org (cookbooks, roles, environments, databags, clients)

click to the client's tab and you can see the list of clients --> for a single object on the right side you can see the three dots (...).
you can click on that and see the delete link. if you click on that delete modal will be open.

βœ… Checklist

πŸ“· Screenshots, if applicable

delete-client.mp4

@netlify
Copy link

netlify bot commented Mar 3, 2021

Deploy preview for chef-automate processing.

Building with commit 7a4f952

https://app.netlify.com/sites/chef-automate/deploys/6041e23a6191b600080a0952

Copy link
Collaborator

@vivekshankar1 vivekshankar1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Working 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>
@kalroy kalroy merged commit 195a4a6 into master Mar 5, 2021
@kalroy kalroy deleted the Vinay/delete_client_UI branch March 5, 2021 10:46
@vinay033
Copy link
Collaborator Author

we need to fix the width of the delete client modal, and I am going to cover in UI improvement issue #4808

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants