London | ITP May-2026 | Dipa Sarker | Sprint 2 | wireframe-final#1328
London | ITP May-2026 | Dipa Sarker | Sprint 2 | wireframe-final#1328Dipa-Sarker wants to merge 1 commit into
Conversation
✅ Deploy Preview for cyf-onboarding-module ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
I encountered some issues with the PR. In the Module-Onboarding repository, both the Form Controls and Wireframe folders exist, which caused confusion and made it difficult to identify the correct assignment folder. To resolve this, I created a new branch named "wireframe-final" to ensure the submission only includes the Wireframe task. |
Dipa-Sarker
added
the
Needs Review
LonMcGregor
added
the
Review in progress
LonMcGregor
left a comment
LonMcGregor
left a comment
There was a problem hiding this comment.
Good start on this task, could you check the criteria again and see my comment about the footer?
| <p> | ||
| This is the default, provided code and no changes have been made yet. | ||
| </p> | ||
| <p>Web Development Basics | README, Wireframe & Git Branches</p> |
There was a problem hiding this comment.
Did you see the criteria about what the footer should do? Check the readme
There was a problem hiding this comment.
Did you mean "Footers Usually Contain Contact Links and Other Information about the site Overall"?
Is this the criteria?
LonMcGregor
added
Reviewed
2 participants
Learners, PR Template
Self checklist
Changelist
Created a webpage using HTML and CSS based on the wireframe. The page explains README files, wireframes, and Git branches.