Skip to content

Revert "fix: improve auth error handling and user feedback" - #124

Open
arzoo0511 wants to merge 1 commit into
mainfrom
revert-98-fix/auth-error-handling
Open

Revert "fix: improve auth error handling and user feedback"#124
arzoo0511 wants to merge 1 commit into
mainfrom
revert-98-fix/auth-error-handling

Conversation

@arzoo0511

Copy link
Copy Markdown
Owner

Reverts #98 issue did not solve the problem

Copilot AI review requested due to automatic review settings May 25, 2026 07:12
@vercel

vercel Bot commented May 25, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ecoplay Error Error May 25, 2026 7:12am

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR reverts the previously added custom auth error-message mapping in AuthContext, returning to passing through Supabase’s raw error.message for register/login failures.

Changes:

  • Removed special-casing of network errors (e.g., “failed to fetch”) during registration.
  • Removed special-casing of invalid-credentials and network errors during login.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@arzoo0511

Copy link
Copy Markdown
Owner Author
image

@arzoo0511 arzoo0511 added help wanted Extra attention is needed level:hard labels May 25, 2026

@arzoo0511 arzoo0511 left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

continuous error auth

@vishnu-varthan1

Copy link
Copy Markdown
Collaborator

please check : Supabase URL and Anon Key environment variables
Supabase Auth URL configuration
Browser Network tab for the signup request
Vercel deployment logs , the auth error handling revert may not be the root cause if the frontend cannot reach the authentication server.

@arzoo0511

Copy link
Copy Markdown
Owner Author

@vishnu-varthan1 can you raise a pr to fix this issue , ill assign it to you

@Anushreebasics Anushreebasics left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

pls resolve conflicts

@arzoo0511

Copy link
Copy Markdown
Owner Author

@CoderOggy78 why did you approve this ? does it work ? can it be merged ?

@Naman-iitm

Copy link
Copy Markdown
Collaborator

please check

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.

6 participants