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

Update_Email_Validation_and_Error_Handling_in_Form #162

Closed

Conversation

smog-root
Copy link
Contributor

@smog-root smog-root commented Oct 16, 2024

Issue_id: NOT CREATED!

Summary: This pull request introduces updates to the form_filling.html file to enhance the email validation functionality. The changes ensure that users can only submit emails from trusted services and provide real-time feedback during form input.

Proof: (Screen recording)! :

https://mega.nz/file/4SUlyRoQ#WQ0Y8FPyRnib9WObm8oZcVXQuHI-UtTS-_K_QtTZnsE


Changes Made:

Email Validation:

  • Implemented a validation check to restrict email submissions to specific trusted domains: Gmail, Yahoo, Outlook, iCloud, and Hotmail.
  • Users will receive immediate feedback if they attempt to enter an email from an untrusted service.

Real-time Error Handling:

An error message is displayed dynamically below the email input field when an invalid email is detected.
The error message disappears when the user enters a valid email address.
The form submission is blocked if the email does not belong to a trusted service, ensuring that only valid submissions proceed.

Note: I did not created an issue so. Assign me this PR, and add gssoc-ext with level3, hacktoberfest-accepted labels!

@Harshdev098

Copy link
Member

@Harshdev098 Harshdev098 left a comment

Choose a reason for hiding this comment

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

Make another PR with the desired changes and different branches, this pr consist of the changes of the previous PR

Closing this PR for now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants