Skip to content

Conversation

@Bentlybro
Copy link
Member

Summary

Improves the user experience when a password reset link has expired or been used, replacing the confusing generic error with a clean, helpful message.

Will be set as a draft for now as i am waiting for a review from design.

Changes

  • Added ExpiredLinkMessage component that displays a user-friendly error state
  • Updated reset password page to detect expired/used links from both:
    • Supabase error format (error=access_denied&error_code=otp_expired&error_description=...)
    • Internal clean format (error=link_expired)
  • Enhanced callback route to detect and map expired/invalid link errors
  • Clear, actionable UI with:
    • Friendly error message explaining what happened
    • "Send Me a New Link" button to request a new reset email
    • Login link for users who already have access

Before

Users saw a confusing URL with error parameters and an unclear form:
http://localhost:3000/reset-password?error=access_denied&error_code=otp_expired&error_description=Email+link+is+invalid+or+has+expired

After

Users see a clean, helpful message explaining the issue and how to fix it.

image image

Checklist 📋

For code changes:

  • I have clearly listed my changes in the PR description
  • I have made a test plan
  • I have tested my changes according to the test plan:
    • Navigate to http://localhost:3000/reset-password?error=access_denied&error_code=otp_expired&error_description=Email+link+is+invalid+or+has+expired and Verify the ExpiredLinkMessage component appears with proper messaging
    • Click "Send Me a New Link" and verify the email form appears

@coderabbitai
Copy link

coderabbitai bot commented Nov 14, 2025

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch bently/secrt-1369-improve-ux-for-expired-or-duplicate-password-reset-links

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@netlify
Copy link

netlify bot commented Nov 14, 2025

Deploy Preview for auto-gpt-docs canceled.

Name Link
🔨 Latest commit 47815a1
🔍 Latest deploy log https://app.netlify.com/projects/auto-gpt-docs/deploys/6917a57116fe310008af9a76

@github-actions github-actions bot added platform/frontend AutoGPT Platform - Front end size/l labels Nov 14, 2025
@netlify
Copy link

netlify bot commented Nov 14, 2025

Deploy Preview for auto-gpt-docs-dev canceled.

Name Link
🔨 Latest commit 47815a1
🔍 Latest deploy log https://app.netlify.com/projects/auto-gpt-docs-dev/deploys/6917a571bb159700084646fe

@deepsource-io
Copy link

deepsource-io bot commented Nov 14, 2025

Here's the code health analysis summary for commits 9c3f679..47815a1. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript✅ SuccessView Check ↗
DeepSource Python LogoPython✅ SuccessView Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Nov 17, 2025
@github-actions
Copy link
Contributor

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

conflicts Automatically applied to PRs with merge conflicts platform/frontend AutoGPT Platform - Front end size/l

Projects

Status: 🆕 Needs initial review
Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants