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

[stable-4.9] Collection upload/deprecate - fix permission checks (#4015) #4664

Merged
merged 1 commit into from
Dec 4, 2023

Conversation

himdel
Copy link
Collaborator

@himdel himdel commented Dec 4, 2023

Backports #4015


Issue: AAH-2439, AAH-2853

Fixes permission checks for collection upload/deprecate on collection list ,namespace detail & collection detail.

action features model or object permissions
copy display_repositories logged in
delete - ansible.delete_collection OR galaxy.change_namespace
remove display_repositories ansible.delete_collection OR galaxy.change_namespace
deprecate - galaxy.change_namespace
upload - galaxy.upload_to_namespace
sign can_create_signatures AND NOT can_upload_signatures galaxy.change_namespace AND galaxy.upload_to_namespace
  • search still won't respect any namespace permissions, ns detail & collection detail will
  • search & ns detail don't do versions, collection detail does

* Collection upload/deprecate - fix permission checks

Issue: AAH-2439
Issue: AAH-2853

* CollectionHeader - move actions & permissions to CollectionDropdown

* Search,NamespaceDetail - use CollectionDropdown

* namespace detail respect namespace permissions

* fix delete collectio/ns disabled message, use description

* inline tryOpenDeleteModalWithConfirm

* deleteFn in all 3 dropdowns

* linter fix

* namespace detail - use object permissions for upload,delete,edit

* tests - constistent kebab-toggle click

* cy.openHeaderKebab - and scroll to top first

* Ratings: don't load scores outside community mode

No-Issue

* collection-kebab - switch to data-cy

* scrollTo fix

* aargh, is it a timing bug?

* upload new version is no longer available without permissions

* openHeaderKebab: add comment about the wait

(cherry picked from commit d0ad573)
@himdel himdel marked this pull request as ready for review December 4, 2023 18:54
@himdel himdel merged commit 32ebc0d into ansible:stable-4.9 Dec 4, 2023
@himdel himdel deleted the 49-4015 branch December 4, 2023 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant