London | 26-ITP-May | Gideon Defar | Sprint 2 | Wireframe#1298
London | 26-ITP-May | Gideon Defar | Sprint 2 | Wireframe#1298gideondefar wants to merge 16 commits into
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.
4 similar comments
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.
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.
-
Your branch is 16 commits behind the CYF's main. Can you update it? Some of the requirements have changed.
-
One of the new requirements is that each article should have a unique image. Can you address this issue?
-
Some of the code is not consistently formatted. Please follow this guide to
enable "Format on Save" in VS Code or use VS Code's "Format Document" feature to keep your code consistently formatted.
…code consistently
|
This is how your current page look like on Netlify:
Its layout does not quite match the wireframe.
This branch is still not yet up to date. Please use the "Sync fork" button to keep it updated.
According to https://validator.w3.org/, there are errors in your code. Can you fix them? |



Learners, PR Template
Self checklist
Changelist
I reviewed and resubmitted the wireframe exercise.