Skip to content
New issue

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

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

Already on GitHub? # to your account

Validate Email Input #10

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

Validate Email Input #10

wants to merge 24 commits into from

Conversation

Ol-create
Copy link
Owner

  • Create variables for form, email, error message.
  • Add check for lowercase
  • Create error function.
  • Add check for validation.
  • Add prevent submit default.
  • Add color to invalid text.

Copy link

@codepantha codepantha left a comment

Choose a reason for hiding this comment

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

Hi @Ol-create 👋,

STATUS APPROVED ✔️ ✔️

Your project is complete! There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Things to highlight 💡:

  • Detailed PR
  • Well implemented UI
  • Meaningful commit messages
  • Descriptive README

Cheers and Happy coding!👏👏👏

Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.


As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

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

Successfully merging this pull request may close these issues.

5 participants