micheaol / Portfolio-project

The purpose of this project is to build a portfolio web app to showcase some if not all of the projects and my skills as a web developer. Built with Javascript, CSS, and HTML.
MIT License
2 stars 0 forks source link

Portfolio-project #11

Closed micheaol closed 3 years ago

micheaol commented 3 years ago

In this pull request I add:

  1. Validation to ensure the content of the email field has to be in lower case.
  2. When the user submits the form, I check if the email is in lower case.
  3. If the validation is OK, the form is sent.
  4. If the validation is not OK, I show an error message to the user near the submit button informing them of the error, and the form is not sent.
netlify[bot] commented 3 years ago

✔️ Deploy Preview for distracted-hermann-695ff6 ready!

🔨 Explore the source changes: 4b2ec08b186c3c6bd13798c347482055be0159de

🔍 Inspect the deploy log: https://app.netlify.com/sites/distracted-hermann-695ff6/deploys/615e6e1e574f030008ad01ab

😎 Browse the preview: https://deploy-preview-11--distracted-hermann-695ff6.netlify.app

netlify[bot] commented 3 years ago

✔️ Deploy Preview for micheaol-portfolio ready!

🔨 Explore the source changes: 4b2ec08b186c3c6bd13798c347482055be0159de

🔍 Inspect the deploy log: https://app.netlify.com/sites/micheaol-portfolio/deploys/615e6e1e4c5bf20007bd9b11

😎 Browse the preview: https://deploy-preview-11--micheaol-portfolio.netlify.app/

Davidosky007 commented 3 years ago

Hi @micheaol 👋,

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

Congratulations! 🎉

nice-job

Cheers and Happy coding!👏👏👏

Feel free to leave any questions or comments in the PR thread if something is not 100% clear.


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.