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

[PRODDEV-219] Implement redirect after login #289

Merged
merged 2 commits into from
Mar 11, 2021
Merged

Conversation

NightHunterSV
Copy link

@NightHunterSV NightHunterSV commented Mar 9, 2021

Related Issue/Ticket:
https://openy.atlassian.net/browse/PRODDEV-219

Steps to test:

  • Visit site as an anonymous

  • Navigate through the site using the main navigation, for example
    image

  • You should be redirected to the login page
    image

  • Log in

  • Ensure you've been redirected to the needed location after the login
    image

  • Change the auth provider and ensure it is also working

Quality checks:

Please check these boxes to confirm this PR covers the following cases:

  • Maintaining our upgrade path is essential. Check one or the other:
    • No updates are necessary for this change.
  • Front end fixes should be tested against all of the Open Y Themes.
    • This change does not contain front-end fixes.
  • I have flagged this PR "Needs Review" or pinged the VY devs/QA
    team in Slack

Copy link

@hamrant hamrant left a comment

Choose a reason for hiding this comment

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

@NightHunterSV let's rename some things in PR

@AnastasiiaPys
Copy link

AnastasiiaPys commented Mar 11, 2021

Tested for dummy, reclique, y-usa providers, personify. Testing passed.

@AnastasiiaPys AnastasiiaPys merged commit 28bc4c7 into 1.3 Mar 11, 2021
@hamrant hamrant deleted the PRODDEV-219 branch March 11, 2021 15:53
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.

5 participants