Uh oh!
There was an error while loading. Please reload this page.
Manchester | 26-ITP-May | Ahmed Elmahmoudi | Sprint 2 | Wireframe - #1400
Manchester | 26-ITP-May | Ahmed Elmahmoudi | Sprint 2 | Wireframe#1400Alaterry8 wants to merge 13 commits into
Conversation
✅ Deploy Preview for cyf-onboarding-module ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Alaterry8
commented
Jun 4, 2026
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.
Alaterry8
commented
Jun 11, 2026
Hi @cjyuan I made the suggested changes, thanks for the review! |
cjyuan
commented
Jun 11, 2026
Code is now consistently formatted. Good job. All done. Note: The "Changelist" header was originally a level 2 header that looks like the Changelist header in your Form-Controls PR. |
illicitonion
commented
Aug 24, 2026
Closing PR because the May ITP run has finished. Feel free to re-open if you're still working on it. |

Self checklist
Change-list: