London | 26-ITP-May | Chun Yan Wong | Sprint 1 | Form Controls

[cywong-dev]

Learners, PR Template

Self checklist

• I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
• My changes meet the requirements of the task
• I have tested my changes
• My changes follow the style guide

Changelist

Display the form for the requirement with validation and display required

Questions

Nil

[netlify]

✅ Deploy Preview for cyf-onboarding-module ready!

Name	Link
🔨 Latest commit	253c27b
🔍 Latest deploy log	https://app.netlify.com/projects/cyf-onboarding-module/deploys/6a2094bf02125b0008703f9d
😎 Deploy Preview	https://deploy-preview-1316--cyf-onboarding-module.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
Lighthouse	2 paths audited Performance: 100 (no change from production) Accessibility: 100 (no change from production) Best Practices: 100 (no change from production) SEO: 86 (no change from production) PWA: - View the detailed breakdown and full score reports
🤖 Make changes	Run an agent on this branch

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[Luro91]

A note on commit messages. It's hard to know for me what happens in a commit if the message is not descriptive. A commit should describe what changed in the code. It's also good practice to split commits into logical blocks (for example for each task or each review comment). This makes it easier to understand and additionally the commits could also be undone more easily if needed.
I like this article about commit messages: https://chris.beams.io/git-commit

[Luro91]

Well done the code works as expected and easy to read.

[illicitonion]

Closing PR because the May ITP run has finished. Feel free to re-open if you're still working on it.