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

[Authenticator] Cannot confirm sign up when using phone + email config and signing up with phone #1765

Closed
1 of 9 tasks
Jordan-Nelson opened this issue Jun 17, 2022 · 5 comments
Assignees
Labels
Authenticator Issues related to the Authenticator UI Component bug Something is not working; the issue has reproducible steps and has been reproduced pending-release Issues that have been addressed in main but have not been released

Comments

@Jordan-Nelson
Copy link
Member

Jordan-Nelson commented Jun 17, 2022

Description

When phone OR email is chosen to be used as the username, and the user signs up with phone number, they are unable to confirm their sign up. The Confirm Sign Up screen shows "email" as the username even though phone number was used during sign up. As a result, the email validator is used. See the screenshot below where the username is a phone number, but the label says "email".

Screen Shot 2022-06-17 at 4 09 58 PM

When this issues is addressed, we should also test the scenario where a user signed up but did not confirm their account, and then later attempts to sign in (which will redirect them to confirm sign up).

Categories

  • Analytics
  • API (REST)
  • API (GraphQL)
  • Auth
  • Authenticator
  • DataStore
  • Storage

Steps to Reproduce

No response

Screenshots

No response

Platforms

  • iOS
  • Android

Android Device/Emulator API Level

No response

Environment

N/A

Dependencies

vNext

Device

N/A

OS

N/A

CLI Version

N/A

Additional Context

No response

@Jordan-Nelson Jordan-Nelson added bug Something is not working; the issue has reproducible steps and has been reproduced Authenticator Issues related to the Authenticator UI Component labels Jun 17, 2022
@Jordan-Nelson
Copy link
Member Author

Note: I am seeing this on /next. I have not confirmed this is present in the latest release.

@Jordan-Nelson
Copy link
Member Author

The linked PR seems to address it. Need to do more testing and confirm if this also needs to be fixed in main.

@Jordan-Nelson
Copy link
Member Author

The issue does exist in main as well.

@Jordan-Nelson Jordan-Nelson self-assigned this Jun 22, 2022
@Jordan-Nelson
Copy link
Member Author

Addressed by #1785

@Jordan-Nelson Jordan-Nelson added the pending-release Issues that have been addressed in main but have not been released label Jun 22, 2022
@Jordan-Nelson
Copy link
Member Author

fixed in the latest version of amplify_authenticator (0.2.1)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Authenticator Issues related to the Authenticator UI Component bug Something is not working; the issue has reproducible steps and has been reproduced pending-release Issues that have been addressed in main but have not been released
Projects
None yet
Development

No branches or pull requests

1 participant