Closed Karyum closed 7 years ago
hey @esraajb we addressed changes 1, 2, 5, 7. can you give some guidance on what num 4 should look like? we will work on testing and sorry about the big pull request.
see app.jsx and routes.jsx here: https://github.com/esraajb/todos-application/tree/master/src
@esraajb we added the label blocked just because we don't want to add testing and routes on this pull request cause it just gonna make it really massive. so can you review the other requested changes so we can start opening issues and pull requests for testing and routes in an effort to keep our pull request smaller.
@esraajb fixed cheers!
read the link above. ill merge this one but make sure to change the discreption for webpack in a seprate PR.
you guys should make sure you understand WHY you use new things and WHAT they does, not only how to implemnt them.
related to #11 close #13