Skip to content

fix(frontend): let users enter password-reset codes - #394

Merged
nourshoreibah merged 2 commits into
mainfrom
cursor/c4518f52
Sep 6, 2026
Merged

nourshoreibah merged 2 commits into
mainfrom
cursor/c4518f52

Conversation

@nourshoreibah

Copy link
Copy Markdown
Collaborator

Summary

  • Cognito emails a verification code (CONFIRM_WITH_CODE), not a reset link. The profile "Send Reset Email" button and the forgot-password page claimed a link was sent and left no place to type the code.
  • Profile now opens a dialog after the email is sent so a signed-in user can enter the code and a new password (they cannot use /reset-password; AuthGate bounces authenticated users off public routes).
  • Forgot-password now collects the code and new password on that page instead of stopping at "Reset Link Sent."

Test plan

  • On Profile, click Send Reset Email, enter the emailed code and a valid new password, and confirm the password changes.
  • On Profile, submit a wrong code and confirm the dialog stays open with an error (password unchanged).
  • From Login → Forgot password, request a code, enter it with a new password, and sign in with the new password.
  • Confirm the emailed message still matches the dialog (code, not a link).

Made with Cursor

Cognito emails a verification code, not a link. The profile
and forgot-password UIs claimed a link was sent and had
nowhere to type the code.

Co-authored-by: Cursor <cursoragent@cursor.com>
@nourshoreibah nourshoreibah added test-environment Creates a temporary (nearly free) test environment. Uses prod DB and cognito no-review The PR review bot won't run labels Aug 30, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🌿 ⏳ Creating preview environment… (logs)

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

Copy link
Copy Markdown
Contributor

🌿 Preview environment — ready ✅

Open: https://d34ajqx96zo93a.cloudfront.net/pr-394/
API: https://am3ulra065.execute-api.us-east-2.amazonaws.com/prod

Shared RDS + Cognito (prod data); DB migrations are not applied here — if this PR adds a migration, endpoints using the new columns will fail until it merges. New commits update this environment in place — a note is posted here on each update. Remove the test-environment label or close the PR to tear it down.

@github-actions

Copy link
Copy Markdown
Contributor

🌿 Test environment updated in place ✅ — Click here to open. updated for 039d82d · logs

@nourshoreibah
nourshoreibah marked this pull request as ready for review September 6, 2026 21:58
@nourshoreibah
nourshoreibah merged commit ef6660a into main Sep 6, 2026
17 checks passed
@nourshoreibah
nourshoreibah deleted the cursor/c4518f52 branch September 6, 2026 21:59
@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

🌿 Preview environment torn down 🧹 — the stack for this PR has been destroyed.

This branch was successfully deployed

1 active deployment
preview — e3233e04 Deployed Sep 6, 2026 by nourshoreibah via teardown #460
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-review The PR review bot won't run test-environment Creates a temporary (nearly free) test environment. Uses prod DB and cognito

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant