Cape Town | 26-ITP-Jan | Pretty Taruvinga | Sprint 2 | Form Controls

[Pretty548]

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

1. Created a form to collect customer name, email, t-shirt colour, and size
2. Added validation to ensure all fields are required
3. Implemented name validation with defined input rules
4. Validated email input using a standard email pattern
5. Provided exactly three colour options and restricted selection to those options
6. Ensured the form has no action attribute as required

Questions

How did you decide what makes a “valid” customer name, and what rules did you use to validate it?

[netlify]

✅ Deploy Preview for cyf-onboarding-module ready!

Name	Link
🔨 Latest commit	2008fe3
🔍 Latest deploy log	https://app.netlify.com/projects/cyf-onboarding-module/deploys/697ccd2ed6229c0008843b80
😎 Deploy Preview	https://deploy-preview-1097--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

---

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

[cjyuan]

Code looks good.

Regarding your question:

How did you decide what makes a “valid” customer name, and what rules did you use to validate it?

I would probably reference the answers gathered from AI and Google search.
If there is a rule, then chances are we can use JavaScript to enforce the rule.

[cjyuan]

Note:
We can also wrap the radio button inside the label element as:

<label><input type="radio" name="color" value="blue"> Blue</label>

This way, we don't have to introduce id and for attributes.

[cjyuan]

The wordings used on lines 32 and 58 could be made more consistent.

May I suggest showing your code to an AI tool and ask it to check for consistency and possible improvements?
It can often help us catch errors, improve consistency, and reinforce best practices.