Skip to content

Fixed issue No #4 (Unable to see error messages during login process on ExamTime website)#19

Open
dhruv389 wants to merge 2 commits into
hereisSwapnil:mainfrom
dhruv389:fix/login-show-error
Open

Fixed issue No #4 (Unable to see error messages during login process on ExamTime website)#19
dhruv389 wants to merge 2 commits into
hereisSwapnil:mainfrom
dhruv389:fix/login-show-error

Conversation

@dhruv389

Copy link
Copy Markdown

Description:
This pull request addresses the issue with the login page not displaying the correct error message when invalid credentials are entered. Currently, the console log shows "Unknown" for this error, while other errors are properly handled and displayed using the Toastify library on the screen.

Changes Made:

Updated the login functionality to handle the "Unknown" error specifically for invalid credentials.
Modified the code to display the appropriate error message using the Toastify library when invalid credentials are entered.

Testing:

Tested the login page with various invalid credential scenarios to ensure the correct error message is displayed.
Verified that the console log no longer shows "Unknown" for invalid credential errors.

closes Issue no : #4

@vercel

vercel Bot commented May 10, 2024

Copy link
Copy Markdown

@dhruv389 is attempting to deploy a commit to the swapnilsingh99's projects Team on Vercel.

A member of the Team first needs to authorize it.

@netlify

netlify Bot commented May 10, 2024

Copy link
Copy Markdown

Deploy Preview for exam-time ready!

Name Link
🔨 Latest commit bf6b8a7
🔍 Latest deploy log https://app.netlify.com/sites/exam-time/deploys/663da23ba76bd60008d7da48
😎 Deploy Preview https://deploy-preview-19--exam-time.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@dhruv389

Copy link
Copy Markdown
Author

Hello @hereisSwapnil,

Apologies for the delay. I've resolved the issue by handling the "Unknown" error, which was previously unaddressed in the main code.

Thank you for your patience.

@vercel

vercel Bot commented May 11, 2024

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
exam-time ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 11, 2024 1:57pm

@hereisSwapnil

Copy link
Copy Markdown
Owner

image

No error message is popped when I put the wrong credentials

@dhruv389

Copy link
Copy Markdown
Author

can you provide to me all values of .env variables of Server side and Client side also , without it i am not able to check.
This is my email_id : drcoder389@gmail.com , please send on it.

@hereisSwapnil

Copy link
Copy Markdown
Owner

can you provide to me all values of .env variables of Server side and Client side also , without it i am not able to check. This is my email_id : drcoder389@gmail.com , please send on it.

Please check .env.dev for keys and credentials

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