-
Notifications
You must be signed in to change notification settings - Fork 113
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
Added revision id for policy file. #5377
Conversation
✔️ Deploy Preview for chef-automate ready! 🔨 Explore the source changes: f6b5dbb 🔍 Inspect the deploy log: https://app.netlify.com/sites/chef-automate/deploys/60ffb9f6df4a6100085cf5c1 😎 Browse the preview: https://deploy-preview-5377--chef-automate.netlify.app |
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 🎉
components/automate-ui/src/app/entities/revisions/revision.reducer.ts
Outdated
Show resolved
Hide resolved
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.
Chnages Looks good, Thanks @himanshi-chhabra
34db94e
to
8d35532
Compare
Signed-off-by: Himanshi Chhabra <hchhabra@progress.com>
Signed-off-by: Himanshi Chhabra <hchhabra@progress.com>
Signed-off-by: Himanshi Chhabra <hchhabra@progress.com>
59d9725
to
f6b5dbb
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Signed-off-by: Himanshi Chhabra hchhabra@progress.com
🔩 Description: What code changed, and why?
⛓️ Related Resources
#5242
👍 Definition of Done
👟 How to Build and Test the Change
PFA Video
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 policy file tab:
...
-> Revisions -> show list of revision if available✅ 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
revision-d.mp4