algovengers / Hairify

https://hairify-ai.vercel.app
16 stars 35 forks source link

Bug: Email Input Type and Error Messaging on Signup and Login Pages , i can add beautiful toast message #129

Open Aniket-2504 opened 6 months ago

Aniket-2504 commented 6 months ago

Hello @algovengers,

I am writing to report a couple of issues on the signup and login pages of the website:

1. Email Input Type Not Defined:

The email input fields on both the signup and login pages are not defined as type "email," which can lead to invalid email formats being entered.

2. Error Messaging:

After entering incorrect information, a small red text appears as an error message. While this indicates an error, it could be improved for a better user experience.

I propose fixing these issues by:

  1. Defining the email input fields as type "email" to ensure proper validation.

  2. Replacing the small red text with a more user-friendly toast message for both error and successful login/signup events.

I am eager to work on this issue and provide a solution that enhances the user experience. Could you please assign this task to me? I am confident that I can deliver a high-quality fix.

Screenshot -

To help illustrate the issue, I have attached a screenshot for your reference.

https://github.com/algovengers/Hairify/assets/145179532/e34bb5d9-022c-44a4-9ca0-9950f8e0ad2a

Thank you for your attention to this matter. I look forward to your response.

github-actions[bot] commented 6 months ago

Hello @Aniket-2504,

Thanks for creating this issue and contributing to Hairify!

We will review your issue soon and provide feedback.

If you're interested in working on this issue, make sure you get yourself assigned before starting your work.

Please ensure that you have followed the contributing guidelines for a smooth review process.

afeefuddin commented 6 months ago

Assigned @Aniket-2504

afeefuddin commented 5 months ago

Are you planning to work on this @Aniket-2504 ?

Aniket-2504 commented 5 months ago

yes bro sorry for that , now i am working on the issue

Aniket-2504 commented 5 months ago

I am done with the issue bro, please check the PR