-
Notifications
You must be signed in to change notification settings - Fork 205
@W-20450671 [Bug Fix] Passwordless Login landingPath, Reset Password landingPath, and Social Login redirectUri value in config not being used #3560
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
Conversation
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
This reverts commit 0228177.
alexvuong
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
yunakim714
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good thanks Jinsu! Is setting the landing path in the config something we already have included in the docs? Or do we need to add that?
@yunakim714 good question. It is for Social Login but not for Passwordless and Password Reset. I'll make sure to include it as part of the docs update story W-20279802 |
Description
Previously the
landingPathandredirectUriset in the config was not being used for creating the routes for passwordless login, reset password, and social login. Instead a constant was being used which meant if the values were changed in the config, the actual routes would not be created.This PR resolves this by ensuring the paths set in the configuration are actually used to generate the routes.
Types of Changes
Changes
landingPath, Reset PasswordlandingPath, and Social LoginredirectUrivalue in config are usedHow to Test-Drive This PR
Disabling the feature does not add routes
landingPathandredirectUri, and social and passwordless login disabled.Enabling the feature and setting custom paths generates the correct routes
/custom-social-callbackwhen redirected back to the storefronthttps://github.com/user-attachments/assets/268c83bd-9d85-48b0-9c50-a6f005bbcb9d
/custom-passwordless-login-landingpathContinue SecurelylandingPathPasswordForgot Password?linkReset PasswordbuttonChecklists
General
Accessibility Compliance
You must check off all items in one of the follow two lists:
or...
Localization