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 intohereisSwapnil: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 intohereisSwapnil:mainfrom
dhruv389:fix/login-show-error

Conversation

@dhruv389
Copy link


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
Copy link

vercel bot commented May 10, 2024

@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
Copy link

netlify bot commented May 10, 2024

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
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
Copy link

vercel bot commented May 11, 2024

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
Owner

image

No error message is popped when I put the wrong credentials

@dhruv389
Copy link
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
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