FACxBeamery / Week3-toDoList-Thomas-Tom

To Do List project
0 stars 1 forks source link

Argument Names #20

Closed Oliversw closed 5 years ago

Oliversw commented 5 years ago

https://github.com/FACxBeamery/Week3-toDoList-Thomas-Tom/blob/79a3bfb9e4daa5ca9b8218c8c86b8335d96b12a6/src/handler.js#L26

I'm sure this is another little copy/paste issue, but I think it's worth changing the name of the argument. In this case convertedData makes less sense to me than something like newTodo or something similar. We want to make our code as easy to read as possible!