Skip to content

login: Switch to correct blog when building destination URL#35

Merged
michaelryanmcneill merged 1 commit intomichaelryanmcneill:masterfrom
jrchamp:fix_login_url
Feb 16, 2018
Merged

login: Switch to correct blog when building destination URL#35
michaelryanmcneill merged 1 commit intomichaelryanmcneill:masterfrom
jrchamp:fix_login_url

Conversation

@jrchamp
Copy link
Copy Markdown
Collaborator

@jrchamp jrchamp commented Feb 15, 2018

Fixes #33.

If any changes are needed, please let me know.

@jrchamp
Copy link
Copy Markdown
Collaborator Author

jrchamp commented Feb 16, 2018

Without the patch, attempting to log in to a site that was not the first one in our list would take us to the first one in our list. The regular user account was sent to an unrelated subsite whereas the super admin account was sent to the root site. After the patch was applied, both accounts were redirected to the correct subsite. We tested reverting and reapplying the patch with the expected results: does not work without this patch, works with this patch.

@michaelryanmcneill michaelryanmcneill merged commit 1c614d3 into michaelryanmcneill:master Feb 16, 2018
@jrchamp jrchamp deleted the fix_login_url branch August 7, 2020 14:16
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.

2 participants