[charlie] [veronica] Todo list and app views created #15
Conversation
There was a problem hiding this comment.
We can update this to use our npm run start command now.
There was a problem hiding this comment.
Same comment about deleting error case and leaving yourself a TODO
|
Done with comments. Lots of comments, but honestly, they are all around the same theme: doc strings and missing semi-colons. I should have pointed them out more often, but I did not want to be nit picky while we were focusing on learning JS, backbone, MVC, making API calls, templating, modularization, npm/gulp scripting, event listening, client side debugging, etc. We did a ton in this PR! I hope you are proud! You've come such a long way. I think we should take a small break from adding features to our app, address these comments, and add some JS unit tests in a followup PR. Unit testing on the client side, I think will be really helpful for us to tackle next. |
2 participants
No description provided.