Manchester | 26-ITP-May | Ahmed Elmahmoudi | Sprint 2 | Wireframe#1400
Manchester | 26-ITP-May | Ahmed Elmahmoudi | Sprint 2 | Wireframe#1400Alaterry8 wants to merge 12 commits into
Conversation
✅ Deploy Preview for cyf-onboarding-module ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
I have to note that just before finishing I noticed that I was doing my work on the main branch, and forgotten to create a separate branch. |
cjyuan
left a comment
There was a problem hiding this comment.
Your code is well-formatted and free of errors, and the page layout closely matches the wireframe. Great job!
-
Could you look up the benefits of using a code formatter? I strongly recommend following this guide
to enable VS Code's "Format on Save" option or use its "Format Document" command. This will help ensure your code is consistently formatted and easier to read and maintain. -
Your PR description is missing a "Changelist section". See this guide for more info.
Note: It's possible to move the commits from main to a different branch. You can try to do that later. Let's focus on completing this PR first.

Self checklist