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

Fix Add/Remove collection ignoring repository object permissions #3602

Merged
merged 1 commit into from
Apr 14, 2023

Conversation

himdel
Copy link
Collaborator

@himdel himdel commented Apr 14, 2023

Issue: AAH-2305

Repository detail > Collection Versions

The permission check wasn't running on object permissions because it didn't know which object permissions to use, fixing.

20230414190221

@github-actions github-actions bot added backport-4.6 This PR should be backported to stable-4.6 (2.3) backport-4.7 This PR should be backported to stable-4.7 (2.4) labels Apr 14, 2023
@himdel himdel removed the backport-4.6 This PR should be backported to stable-4.6 (2.3) label Apr 14, 2023
@himdel himdel force-pushed the 2305-repo-perm branch 2 times, most recently from bb74d91 to 01dd284 Compare April 14, 2023 19:24
@himdel himdel merged commit 02c7968 into ansible:master Apr 14, 2023
@himdel himdel deleted the 2305-repo-perm branch April 14, 2023 23:11
@patchback
Copy link

patchback bot commented Apr 14, 2023

Backport to stable-4.7: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-4.7/02c7968f83592803c19c0993b37bb4fb4c8c453b/pr-3602

Backported as #3605

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Apr 14, 2023
himdel added a commit that referenced this pull request Apr 15, 2023
…) (#3605)

Issue: AAH-2305
(cherry picked from commit 02c7968)

Co-authored-by: Martin Hradil <[email protected]>
@github-actions github-actions bot added the backported-4.7 This PR has been backported to stable-4.7 (2.4) label Apr 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-4.7 This PR should be backported to stable-4.7 (2.4) backported-4.7 This PR has been backported to stable-4.7 (2.4)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant