Skip to content

London | 26-ITP-May | Gideon Defar | Sprint 1 | Form Control#1388

Open
gideondefar wants to merge 13 commits into
CodeYourFuture:mainfrom
gideondefar:feature/form-control
Open

London | 26-ITP-May | Gideon Defar | Sprint 1 | Form Control#1388
gideondefar wants to merge 13 commits into
CodeYourFuture:mainfrom
gideondefar:feature/form-control

Conversation

@gideondefar
Copy link
Copy Markdown

Self checklist

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

Changelist
Text input for name, and email input for email with proper validation. Select inputs for color and size with pre-defined choices. Validated with Lighthouse.

@netlify
Copy link
Copy Markdown

netlify Bot commented Jun 1, 2026

Deploy Preview for cyf-onboarding-module ready!

Name Link
🔨 Latest commit 4986b40
🔍 Latest deploy log https://app.netlify.com/projects/cyf-onboarding-module/deploys/6a1cee730106da00088b8d8a
😎 Deploy Preview https://deploy-preview-1388--cyf-onboarding-module.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
2 paths audited
Performance: 100 (no change from production)
Accessibility: 100 (no change from production)
Best Practices: 100 (no change from production)
SEO: 91 (🟢 up 5 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.

@gideondefar gideondefar added 📅 Sprint 1 Assigned during Sprint 1 of this module Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Jun 1, 2026
@gideondefar
Copy link
Copy Markdown
Author

My pull request was labeled as completed, but there was a conflict of commits between my local and remote branches, where some commits were behind. I tried to sync fork, but it didn't work; then I merged the branch with the new PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. 📅 Sprint 1 Assigned during Sprint 1 of this module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant