-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Description
README
This was very detailed- good job!
User stories
Have you met the user stories?
Mostly- very interesting and fun to use!
Next step: work on responsiveness

Learning outcomes
Have you met the LOs?
Yes!
UI bugs
Any bugs?
At one point, it stopped working. Not sure why (got this error: Error: Cannot read properties of undefined (reading 'translated')) even though I put things in. Maybe make it clearer.

Instructions
Instructions were very straight forward.
File structure
FS, straight forward (not sure why css was in same folder as js? but clear apart from that).
Flow of control
Code was clear to follow.
Naming
Variables and functions named clearly.
Readability
Very readable code. So sorry about ours!!
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels