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 proxy client details page UI #4008

Merged
merged 6 commits into from
Jul 14, 2020
Merged

Conversation

vinay033
Copy link
Collaborator

🔩 Description: What code changed, and why?

Introduced client details page view.

⛓️ Related Resources

#3489

👍 Definition of Done

Client details page of infra view where we are displaying the client related details and public key tab.

👟 How to Build and Test the Change

STEP 1
inside the hab studio

[default:/src:0]# build components/automate-ui-devproxy/
[default:/src:0]# start_automate_ui_background
[default:/src:0]# start_all_services

STEP 2
open new window
go to automate UI path

$ cd components/automate-ui
and run the command 

npm run serve:hab

To add data https://github.com/chef/automate/blob/master/dev-docs/adding-data/adding_test_data.md#adding-data-to-infra-views
Go to Infrastructure tab >> Chef Servers it's under Chef-Server feature flag.
Click on Server list >> Organisation list >> clients Tab >> then click on any of the client list items

✅ Checklist

📷 Screenshots

client-details

@vinay033 vinay033 self-assigned this Jun 30, 2020
@vinay033 vinay033 linked an issue Jun 30, 2020 that may be closed by this pull request
@vjeffrey vjeffrey requested a review from a team July 2, 2020 19:02
Copy link
Contributor

@tarablack01 tarablack01 left a comment

Choose a reason for hiding this comment

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

Good Job 🎉

@vinay033 vinay033 force-pushed the Vinay/client_details_view branch from 995ebc2 to 82261dd Compare July 8, 2020 11:55
@vinay033 vinay033 requested a review from SEAjamieD July 8, 2020 11:56
Copy link
Contributor

@SEAjamieD SEAjamieD left a comment

Choose a reason for hiding this comment

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

🎉

@vjeffrey vjeffrey merged commit e8e49e2 into master Jul 14, 2020
@chef-expeditor chef-expeditor bot deleted the Vinay/client_details_view branch July 14, 2020 01:54
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.

Infra proxy client details page view
4 participants