-
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
removed id from details page and edit data bag pop-up. #5249
Conversation
👷 Deploy Preview for chef-automate processing. 🔨 Explore the source changes: 9acb657 🔍 Inspect the deploy log: https://app.netlify.com/sites/chef-automate/deploys/60d1b26194fbc60007adf1c1 |
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.
👍🏻
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.
Signed-off-by: Himanshi Chhabra <hchhabra@progress.com>
Signed-off-by: Himanshi Chhabra <hchhabra@progress.com>
b28847a
to
a7b35c3
Compare
Signed-off-by: Himanshi Chhabra <hchhabra@progress.com>
Kudos, SonarCloud Quality Gate passed!
|
Signed-off-by: Himanshi Chhabra hchhabra@progress.com
🔩 Description: What code changed, and why?
In the Data bag item, the id is editable and getting API error so removed the id.
⛓️ Related Resources
#5024
👍 Definition of Done
Removed the ID from the details page of data bag item and inside edit pop-up
👟 How to Build and Test the Change
To add data https://github.com/chef/automate/blob/master/dev-docs/adding-data/adding_test_data.md#adding-data-to-infra-views
Steps for Databag Item tab:
✅ Checklist
All PRs must tick these:
With occasional exceptions, all PRs from Progress employees must tick these:
make spell
in any component directory)All PRs from Progress employees should tick these if appropriate:
Please add a note next to any checkbox above if you are NOT ticking it.
📷 Screenshots, if applicable