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 dispatch page filter param - use github_user for legacy roles #4144

Merged
merged 1 commit into from
Aug 27, 2023

Conversation

himdel
Copy link
Collaborator

@himdel himdel commented Aug 24, 2023

Issue: AAH-2670
(Introduced in #4090)

?username filter doesn't work on the legacy role API, use ?github_user instead.

@github-actions github-actions bot added the backport-4.7 This PR should be backported to stable-4.7 (2.4) label Aug 24, 2023
@himdel himdel added the community comunity mode feature label Aug 24, 2023
@himdel himdel merged commit 271a9d5 into ansible:master Aug 27, 2023
@himdel himdel deleted the aah2670 branch August 27, 2023 00:06
@patchback
Copy link

patchback bot commented Aug 27, 2023

Backport to stable-4.7: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-4.7/271a9d592d19635a7e0b24d98511d114a8a8fdce/pr-4144

Backported as #4147

🤖 @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 Aug 27, 2023
himdel added a commit that referenced this pull request Aug 27, 2023
) (#4147)

Issue: AAH-2670
(cherry picked from commit 271a9d5)

Co-authored-by: Martin Hradil <mhradil@redhat.com>
@github-actions github-actions bot added the backported-4.7 This PR has been backported to stable-4.7 (2.4) label Aug 27, 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) community comunity mode feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants