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

Users Who Did Not Verify on Registration Should Be Redirected to ConfirmSignUp Page #6295

Open
2 tasks
rajleechaudry opened this issue Jan 13, 2025 · 5 comments
Labels
feature-request Request a new feature

Comments

@rajleechaudry
Copy link

On which framework/platform would you like to see this feature implemented?

React Native

Which UI component is this feature-request for?

Authenticator

Please describe your feature-request in detail.

If a user registers through Amplify Cognito, but doesn't immediately verify their confirmation code, then tries to come back later and login, they should be auto redirected to the ConfirmSignUp. Amplify itself returns the response on signIn of {"isSignedIn": false, "nextStep": {"signInStep": "CONFIRM_SIGN_UP"}}

But the flow does not get redirected. This is pretty odd, and works in the SignUp flow. I.E. When a user signs up, they get redirected to the ConfirmSignUp.

Please describe a solution you'd like.

Unverified User Tries to signIN -> some way to redirect them to ConfirmSignUp component w/o doing a hack like leaving the Amplify auth navigator to put up my own confirmation page, and then redirect back to the Authenticator. There should be a setting or it should be automatic that Amplify follows it's own returned next step "CONFIRM_SIGN_UP"

We love contributors! Is this something you'd be interested in working on?

  • 👋 I may be able to implement this feature request.
  • ⚠️ This feature might incur a breaking change.
@rajleechaudry rajleechaudry added the feature-request Request a new feature label Jan 13, 2025
@github-actions github-actions bot added pending-triage Issue is pending triage pending-maintainer-response Issue is pending response from an Amplify UI maintainer labels Jan 13, 2025
@cwomack cwomack removed the pending-triage Issue is pending triage label Jan 14, 2025
@cwomack
Copy link
Member

cwomack commented Jan 14, 2025

Hello, @rajleechaudry 👋. It sounds like you're asking for the immediate rendering of the "confirm sign in" screen to happen upon app launch in situations where a user wasn't able to complete the sign up flow, but then comes back later to the app. Is that correct?

With this being opened as a feature request and that being a departure from the currently functionality, just want to confirm the ask here. Thanks!

@cwomack cwomack added pending-community-response Issue is pending response from the issue requestor or other community members and removed pending-maintainer-response Issue is pending response from an Amplify UI maintainer labels Jan 14, 2025
@rajleechaudry
Copy link
Author

rajleechaudry commented Jan 15, 2025 via email

@github-actions github-actions bot added pending-maintainer-response Issue is pending response from an Amplify UI maintainer and removed pending-community-response Issue is pending response from the issue requestor or other community members labels Jan 15, 2025
@cwomack
Copy link
Member

cwomack commented Jan 15, 2025

@rajleechaudry, thanks for the reply and additional context. I want to make sure we're not potentially talking about an existing feature request (and poor user experience) that is well known and referenced in this issue on the amplify-js repo where users can get into a "deadlocked" state when they haven't confirmed their account with Cognito, but have attempted to sign up.

Does that issue capture the situations you're describing? If not, then can you help us understand the difference in the ask here? If we can consolidate this feature request into that issue and get another "+1" of another customer experiencing this blocker that requires Cognito implementation to change, then it helps us gain traction. Thanks!

@cwomack cwomack added pending-community-response Issue is pending response from the issue requestor or other community members and removed pending-maintainer-response Issue is pending response from an Amplify UI maintainer labels Jan 15, 2025
@ysabri
Copy link

ysabri commented Jan 23, 2025

I'm commenting to clarify one thing because I do believe it could be a similar request.

My issue was filed in V5 which could cause confusion given that a lot of the API and hub events changed since.

I do ask for what they ask for here: aws-amplify/amplify-js#11270 (comment)

One thing @rajleechaudry hasn't clarified, is whether they set the autoSignIn flag to true when calling the signUp method. We do this and end up with accounts that are Confirmed and have the email User Attribute as Not verified. In that case, there is no way for us to trigger the email verification flow in the Authenticator.

If we were not to set the autoSignIn flag when calling signUp, back in v5, the Authenticator will go into the confirm signup flow which on success will set the user as Confirmed and email User Attribute as Verified.

Meaning, in the Authenticator, for unauthenticated users, there is no way to trigger the email User Attribute verification flow if their account is marked as Confirmed.

@github-actions github-actions bot added pending-maintainer-response Issue is pending response from an Amplify UI maintainer and removed pending-community-response Issue is pending response from the issue requestor or other community members labels Jan 23, 2025
@jordanvn
Copy link
Member

Hi @ysabri, thanks for providing additional information.

We do this and end up with accounts that are Confirmed and have the email User Attribute as Not verified.

This sounds like a potential bug, so I have opened #6316 to bring it to attention for triaging.

@github-actions github-actions bot removed the pending-maintainer-response Issue is pending response from an Amplify UI maintainer label Jan 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request Request a new feature
Projects
None yet
Development

No branches or pull requests

4 participants