Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

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

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.

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.

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 : [email protected] , 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 : [email protected] , 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