-
Notifications
You must be signed in to change notification settings - Fork 0
Description
README
[Does the README describe the project using the subheadings: Why?, What?, How? i.e. Why have you created this repo, what does it do, how does it do it?]
There is no user journey, Project overview, no Project Milestones.
User stories
[Does the project meet the user stories for that week?]
Yes, looks great.
Learning outcomes
[Does it demonstrate the learning outcomes for that week?]
Yes, there are two Testing files.
There are a lot of comments codes that you might want to delete.
UI bugs
[Can you see any obvious bugs or areas to improve?]
Drag and drop from "In-progress" to "Complete" doesn't work.
Drag and drop from New folder to "Complete" doesn't work.
Responsive design- problem with margin right - "Complete" section
Instructions
[Does everything work as expected or were there missing instructions?]
Same as above problems.
File structure
[When you open the project in your editor, does the file structure make sense?]
Yes
Flow of control
[ Can you you follow the different paths the code might take?]
Yes
Naming
[Do variables and functions have clear and descriptive names?]
Yes
Readability
[Do you understand the code?]
yes
Well done.