-
Notifications
You must be signed in to change notification settings - Fork 130
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
[Login] Fix issue of showing the password fallback button for passwordless accounts #13201
Conversation
To make sure the password fallback button is hidden when the account is passwordless
📲 You can test the changes from this Pull Request in WooCommerce-Wear Android by scanning the QR code below to install the corresponding build.
|
📲 You can test the changes from this Pull Request in WooCommerce Android by scanning the QR code below to install the corresponding build.
|
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.
Thanks for addressing this issue, @hichamboushaba! I have a question. I was able to trigger magic link with my account that has a password, and the "Log in with magic link" button was useful in this case. This PR removes the button not only for passwordless accounts but for all accounts. Was this intentional?
Still, a user with a password can navigate back from the magic link screen to the password screen by tapping the back button. Do you think this is fine, or should we check isPasswordless
state and remove the "Log in with magic link" button based on that?
before | after |
---|---|
Thanks @irfano for the review, and sorry for the late reply. Whether the button should be shown or not is a different discussion, personally I think it shouldn't because the user intentionally opened the screen and can use the back button to revert his decision. But my opinion is not a strong one, so if you believe we should change the behavior, we can start a discussion separately; the discussion also allows us to keep the platforms aligned (iOS doesn't show it either). |
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.
Thanks for the explanation, Hicham. Then we can proceed with merging this and aligning the platforms. LGTM! 👍🏻
Closes: #13187
Description
This PR fixes a long standing bug that we had in the app, since the introduction of the magic link experiment more than 2 years ago, we accidentally ignored the
allowPassword
argument, and this issue persisted after removing the experiment and also after introducing QR code scanning feature.Steps to reproduce
/v1.1/users/%/auth-options
to return{"passwordless": true, "email_verified": true }
)Testing information
Confirm that the button "Or log in with password" is hidden.
The tests that have been performed
^
RELEASE-NOTES.txt
if necessary. Use the "[Internal]" label for non-user-facing changes.Reviewer (or Author, in the case of optional code reviews):
Please make sure these conditions are met before approving the PR, or request changes if the PR needs improvement: