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 · 3 comments
Labels
feature-request Request a new feature pending-community-response Issue is pending response from the issue requestor or other community members

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
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 pending-community-response Issue is pending response from the issue requestor or other community members
Projects
None yet
Development

No branches or pull requests

2 participants