London | 26-ITP-May | Martin Mwaka | Sprint 1 | Form Controls#1254
London | 26-ITP-May | Martin Mwaka | Sprint 1 | Form Controls#1254Temceo wants to merge 8 commits into
Conversation
✅ Deploy Preview for cyf-onboarding-module ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Your PR description is incomplete. You filled out a template (that starts "Learners, PR Template") when you created this PR - you can see it at the top of this page. Make sure to fill in all fields in the template. Please ensure:
If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above. |
|
Your PR description is incomplete. You filled out a template (that starts "Learners, PR Template") when you created this PR - you can see it at the top of this page. Make sure to fill in all fields in the template. Please ensure:
If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above. |
|
If you're having difficulty with the automated PR checks please reach out in slack.
|
|
Your PR description is incomplete. You filled out a template (that starts "Learners, PR Template") when you created this PR - you can see it at the top of this page. Make sure to fill in all fields in the template. Please ensure:
If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above. |
|
Your PR description is incomplete. You filled out a template (that starts "Learners, PR Template") when you created this PR - you can see it at the top of this page. Make sure to fill in all fields in the template. Please ensure:
If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above. |
1 similar comment
|
Your PR description is incomplete. You filled out a template (that starts "Learners, PR Template") when you created this PR - you can see it at the top of this page. Make sure to fill in all fields in the template. Please ensure:
If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above. |
|
Your PR description is incomplete. You filled out a template (that starts "Learners, PR Template") when you created this PR - you can see it at the top of this page. Make sure to fill in all fields in the template. Please ensure:
If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above. |
2 similar comments
|
Your PR description is incomplete. You filled out a template (that starts "Learners, PR Template") when you created this PR - you can see it at the top of this page. Make sure to fill in all fields in the template. Please ensure:
If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above. |
|
Your PR description is incomplete. You filled out a template (that starts "Learners, PR Template") when you created this PR - you can see it at the top of this page. Make sure to fill in all fields in the template. Please ensure:
If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above. |
|
Your PR description is incomplete. You filled out a template (that starts "Learners, PR Template") when you created this PR - you can see it at the top of this page. Make sure to fill in all fields in the template. Please ensure:
If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above. |
|
Your PR description is incomplete. You filled out a template (that starts "Learners, PR Template") when you created this PR - you can see it at the top of this page. Make sure to fill in all fields in the template. Please ensure:
If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above. |
|
Your PR description is incomplete. You filled out a template (that starts "Learners, PR Template") when you created this PR - you can see it at the top of this page. Make sure to fill in all fields in the template. Please ensure:
If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above. |
LonMcGregor
left a comment
There was a problem hiding this comment.
Good start on the form controls task, see my comment for one thing to improve.
Could you fix the submission so that it does not contain the changes to the wireframe task please? We want submissions to only cover one specific task at a time.
| <div class="form-group"> | ||
| <!-- Add input for name - required and validate --> | ||
| <label for="name">Name</label> | ||
| <input type="text" id="name" placeholder="Joe Bloggs" required> |
There was a problem hiding this comment.
Does the task criteria mention something about a minimum length?
|
Your PR description is incomplete. You filled out a template (that starts "Learners, PR Template") when you created this PR - you can see it at the top of this page. Make sure to fill in all fields in the template. Please ensure:
If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above. |
1 similar comment
|
Your PR description is incomplete. You filled out a template (that starts "Learners, PR Template") when you created this PR - you can see it at the top of this page. Make sure to fill in all fields in the template. Please ensure:
If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above. |
|
Your PR description is incomplete. You filled out a template (that starts "Learners, PR Template") when you created this PR - you can see it at the top of this page. Make sure to fill in all fields in the template. Please ensure:
If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above. |
1 similar comment
|
Your PR description is incomplete. You filled out a template (that starts "Learners, PR Template") when you created this PR - you can see it at the top of this page. Make sure to fill in all fields in the template. Please ensure:
If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above. |
|
Your PR description is incomplete. You filled out a template (that starts "Learners, PR Template") when you created this PR - you can see it at the top of this page. Make sure to fill in all fields in the template. Please ensure:
If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above. |
|
Your PR description is incomplete. You filled out a template (that starts "Learners, PR Template") when you created this PR - you can see it at the top of this page. Make sure to fill in all fields in the template. Please ensure:
If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above. |
|
Your PR description is incomplete. You filled out a template (that starts "Learners, PR Template") when you created this PR - you can see it at the top of this page. Make sure to fill in all fields in the template. Please ensure:
If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above. |
|
Your PR description is incomplete. You filled out a template (that starts "Learners, PR Template") when you created this PR - you can see it at the top of this page. Make sure to fill in all fields in the template. Please ensure:
If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above. |
|
Your PR description is incomplete. You filled out a template (that starts "Learners, PR Template") when you created this PR - you can see it at the top of this page. Make sure to fill in all fields in the template. Please ensure:
If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above. |
|
Your PR description is incomplete. You filled out a template (that starts "Learners, PR Template") when you created this PR - you can see it at the top of this page. Make sure to fill in all fields in the template. Please ensure:
If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above. |
1 similar comment
|
Your PR description is incomplete. You filled out a template (that starts "Learners, PR Template") when you created this PR - you can see it at the top of this page. Make sure to fill in all fields in the template. Please ensure:
If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above. |

Self checklist
Changelist
Briefly explain your PR.
I have forked the original form controls and made updates on a branch (sprint1_control_forms).
I have add a new form using semantic html with labels and controls for entry of name, email, colour and sizes