London | ITP-MAY-2026 | Ebrahim Moqbel | sprint 1 | Wireframe to Web code#1060
London | ITP-MAY-2026 | Ebrahim Moqbel | sprint 1 | Wireframe to Web code#1060Ebrahim-Moqbel wants to merge 23 commits intoCodeYourFuture:mainfrom
Conversation
✅ Deploy Preview for cyf-onboarding-module ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
images for the website
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
cjyuan
left a comment
There was a problem hiding this comment.
A well-prepared PR makes it easier for reviewers to approve it with minimal back-and-forth.
Can you take a look at this PR Guide and General Feedback to see if there
is anything you can do to make your PR more robust and ready?
Doing so can help speed up the review process.
|
There isn't any new commits on this PR branch since I left a comment 4 days ago. Did you forget to push your changes to GitHub? The commit you made to the cloned repo on your computer are not sync to GitHub automatically, you will need to manually sync/push the changes to GitHub. |
Thank you for your comment I believe I have done all the tasks can you spot me what I need to do. Thank you |
|
According to https://validator.w3.org/, there are errors in your code. Can you fix them? |
|
Changes look good. Please note that in CYF courses, the recommended way to inform the reviewer of your changes is to do both of the following:
|
|
Closing PR because the January ITP run has finished. Feel free to re-open if you're still working on it. |

Learners, PR Template
Self checklist
Changelist
I have made the changes and updated the contents and by PR this commits
Questions