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 #12815 Direct links to AWX do not reroute the user after authentication #14399

Merged
merged 1 commit into from
Oct 3, 2023

Conversation

Sasa993
Copy link
Contributor

@Sasa993 Sasa993 commented Aug 31, 2023

SUMMARY

This fixes an issue with wrong user redirection after login.

ISSUE TYPE
  • Bug, Docs Fix or other nominal change
COMPONENT NAME
  • UI
AWX VERSION
22.7.1.dev74+g4b78b196aa
ADDITIONAL INFORMATION

Before the change, user would be redirected to the dashboard instead of the original link.
Conversation on this topic: https://groups.google.com/g/awx-project/c/SdAtDEcn4m4


@fosterseth
Copy link
Member

@mabashian is there someone from UI team who can review this community PR? thanks!

@marshmalien marshmalien self-requested a review September 20, 2023 19:42
@marshmalien
Copy link
Member

@Sasa993 👋 Thank you for the contribution! This appears to resolve the link redirect issue. Do you mind rebasing this branch with the latest devel? I'll give it another look once it is up-to-date and see if we can get this merged 🚢

@Sasa993
Copy link
Contributor Author

Sasa993 commented Sep 22, 2023

Hi @marshmalien 👋
Thank you for taking the time to review the PR.
Rebasing done 😌

@vishsanghishetty vishsanghishetty self-assigned this Oct 2, 2023
@vishsanghishetty
Copy link
Contributor

QE Verified, merging 🚀

…cation

Signed-off-by: Sasa993 <jovicic.sasa@hotmail.com>
Copy link
Member

@marshmalien marshmalien left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@vishsanghishetty vishsanghishetty merged commit 9d60b0b into ansible:devel Oct 3, 2023
@Sasa993
Copy link
Contributor Author

Sasa993 commented Oct 3, 2023

Thank you @marshmalien and @vishsanghishetty 🚀

@marshmalien
Copy link
Member

Thank YOU @Sasa993 😃

Sasa993 pushed a commit to Sasa993/awx that referenced this pull request Oct 26, 2023
Sasa993 pushed a commit to Sasa993/awx that referenced this pull request Oct 26, 2023
Sasa993 pushed a commit to Sasa993/awx that referenced this pull request Oct 27, 2023
…#14399)

Signed-off-by: Sasa Jovicic <sjovicic@anexia-it.com>
Sasa993 pushed a commit to Sasa993/awx that referenced this pull request Nov 9, 2023
…#14399)

Signed-off-by: Sasa Jovicic <sjovicic@anexia-it.com>
Sasa993 pushed a commit to Sasa993/awx that referenced this pull request Nov 15, 2023
…#14399)

Signed-off-by: Sasa Jovicic <sjovicic@anexia-it.com>
Sasa993 pushed a commit to Sasa993/awx that referenced this pull request Nov 20, 2023
…#14399)

Signed-off-by: Sasa Jovicic <sjovicic@anexia-it.com>
Sasa993 pushed a commit to Sasa993/awx that referenced this pull request Jan 15, 2024
…#14399)

Signed-off-by: Sasa Jovicic <sjovicic@anexia-it.com>
dmzoneill pushed a commit to Sasa993/awx that referenced this pull request Feb 15, 2024
…#14399)

Signed-off-by: Sasa Jovicic <sjovicic@anexia-it.com>
Sasa993 pushed a commit to Sasa993/awx that referenced this pull request Feb 15, 2024
…#14399)

Signed-off-by: Sasa Jovicic <sjovicic@anexia-it.com>
Sasa993 pushed a commit to Sasa993/awx that referenced this pull request Feb 19, 2024
…#14399)

Signed-off-by: Sasa Jovicic <sjovicic@anexia-it.com>
dmzoneill pushed a commit to Sasa993/awx that referenced this pull request Feb 20, 2024
…#14399)

Signed-off-by: Sasa Jovicic <sjovicic@anexia-it.com>
dmzoneill pushed a commit that referenced this pull request Feb 20, 2024
Signed-off-by: Sasa Jovicic <sjovicic@anexia-it.com>
Sasa993 pushed a commit to Sasa993/awx that referenced this pull request Feb 22, 2024
…#14399)

Signed-off-by: Sasa Jovicic <sjovicic@anexia-it.com>
djyasin pushed a commit to djyasin/awx that referenced this pull request Sep 16, 2024
…uthentication (ansible#14399)

Signed-off-by: Sasa993 <jovicic.sasa@hotmail.com>
Co-authored-by: Sasa Jovicic <sjovicic@anexia-it.com>
djyasin pushed a commit to djyasin/awx that referenced this pull request Sep 16, 2024
…#14399)

Signed-off-by: Sasa Jovicic <sjovicic@anexia-it.com>
djyasin pushed a commit to djyasin/awx that referenced this pull request Nov 11, 2024
…uthentication (ansible#14399)

Signed-off-by: Sasa993 <jovicic.sasa@hotmail.com>
Co-authored-by: Sasa Jovicic <sjovicic@anexia-it.com>
djyasin pushed a commit to djyasin/awx that referenced this pull request Nov 11, 2024
…#14399)

Signed-off-by: Sasa Jovicic <sjovicic@anexia-it.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants