You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The project has been tested by a some mentor(s) before we run this for the first time with trainees. We will also collect feedback after we run it with trainees. For now, this task contains a dump of any feedback collected so far.
Items in here are welcome to be discussed or picked up. Note that any changes made now can be merged to the project template repo, but won't end up in the trainees projects (unless they actively decide to sync with the parent fork, or we find a major issue that we need to request trainees merge in).
Overall, I think the structure is strong, and the progression goes step by step with a clear guideline.
From a frontend perspective, one thing I noticed is that previously we were introduced to using frameworks like Next and UI libraries, so there wasn’t much effort needed on styling. In this project, everything is built with plain React and CSS, so I had to handle layout and styling myself. This is great for learning core frontend skills, but at the same time, without a predefined structure or design reference, it felt a bit unclear at the beginning how to approach the overall page layout and keep the UI consistent.
Also, some files already include TODOs for later weeks, so it’s not always very clear what to focus on vs ignore at the current stage. It still works, but requires a bit of interpretation when following the steps.
The project has been tested by a some mentor(s) before we run this for the first time with trainees. We will also collect feedback after we run it with trainees. For now, this task contains a dump of any feedback collected so far.
Items in here are welcome to be discussed or picked up. Note that any changes made now can be merged to the project template repo, but won't end up in the trainees projects (unless they actively decide to sync with the parent fork, or we find a major issue that we need to request trainees merge in).
Feedback from Jianxin on slack (thread):