-
Notifications
You must be signed in to change notification settings - Fork 170
ROX-32557: Update Base Images permission to ImageAdministration #18415
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
Conversation
|
Skipping CI for Draft Pull Request. |
|
Images are ready for the commit at 64b712a. To use with deploy scripts, first |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #18415 +/- ##
==========================================
- Coverage 48.93% 48.93% -0.01%
==========================================
Files 2631 2631
Lines 197964 197964
==========================================
- Hits 96872 96868 -4
- Misses 93703 93708 +5
+ Partials 7389 7388 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
7b11122 to
623aa05
Compare
|
Tests should be resolved once #18422 goes in. The reason is explained in this comment: #18422 (comment) |
Signed-off-by: Saif Chaudhry <[email protected]>
Signed-off-by: Saif Chaudhry <[email protected]>
Signed-off-by: Saif Chaudhry <[email protected]>
623aa05 to
64b712a
Compare
Description
Updates the UI permission check for the Base Images route from
ImagetoImageAdministrationto align with the backend.Additionally, restricts modification actions (add, edit, delete) to users with write access for
ImageAdministration. Users with read-only access will see a read-only view of the base images table.User-facing documentation
Testing and quality
ROX_BASE_IMAGE_DETECTIONfeature flagAutomated testing
How I validated my change
Screenshots
No Read/Write Access
Read Access Only
Read/Write Access