Skip to content

Comments

fix: redirect user to sign-in page after successful signup#408

Open
souma9830 wants to merge 1 commit intoMuneerali199:mainfrom
souma9830:fix/signup-success-redirect
Open

fix: redirect user to sign-in page after successful signup#408
souma9830 wants to merge 1 commit intoMuneerali199:mainfrom
souma9830:fix/signup-success-redirect

Conversation

@souma9830
Copy link

@souma9830 souma9830 commented Feb 11, 2026

Summary

This PR fixes an authentication flow bug where users are redirected to a generic
“Something went wrong” page after successfully creating an account.

Instead, users are now redirected to the Sign In page after successful registration.


Problem


Changes Made

  • Corrected sign-up success handling logic
  • Ensured successful registration triggers a redirect to the Sign In page
  • Prevented false error-state navigation after successful account creation
  • Improved overall authentication flow consistency

Expected Behavior After Fix

  • User completes Sign Up successfully
  • Optional success feedback is shown
  • User is redirected to the Sign In page
  • No erroneous “Something went wrong” page appears

Impact

  • Improves first-time user experience
  • Prevents confusion and duplicate sign-ups
  • Aligns authentication flow with standard UX practices

Screenshots

Before----

Screenshot 2026-02-12 011550

Checklist

  • Signup success redirect fixed
  • No breaking changes
  • Tested locally

Please review and let me know if any improvements are required 🙌

Uploading Recording 2026-02-12 012047 (1).mp4…

@vercel
Copy link

vercel bot commented Feb 11, 2026

@souma9830 is attempting to deploy a commit to the muneerali199's projects Team on Vercel.

A member of the Team first needs to authorize it.

@netlify
Copy link

netlify bot commented Feb 11, 2026

👷 Deploy request for docmagic1 pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit f894e0e

@netlify
Copy link

netlify bot commented Feb 11, 2026

👷 Deploy request for docmagic-muneer pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit f894e0e

Copy link
Author

@souma9830 souma9830 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@Muneerali199
Copy link
Owner

hey @souma9830 ,show me video of your work as video is not uploaded properly .

@souma9830
Copy link
Author

If you satisfied You can Merge it sir

@souma9830
Copy link
Author

Merge it

@souma9830
Copy link
Author

I think sir badge should be OSCG26 not Oscg 26

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants