paulsaenzsucre/To-Do-List

Code review 2022-12-13

Closed this issue 2 years ago · 0 comments

paulsaenzsucre commented 2 years ago

You did a fantastic job so far!!!! There are some issues that you still need to work, but you are almost there:

📌 These blocks of code are the same, you can refactor as a function to DRY:
https://github.com/elafildecolor/To-Do-List/blob/8cec725ddb451b0a1b6095b55897a2fdb59754f4/modules/TaskPresenter.js#L98-L100

https://github.com/elafildecolor/To-Do-List/blob/8cec725ddb451b0a1b6095b55897a2fdb59754f4/modules/TaskPresenter.js#L104-L106

📌 And these two either:
https://github.com/elafildecolor/To-Do-List/blob/8cec725ddb451b0a1b6095b55897a2fdb59754f4/modules/TaskRepository.js#L33-L36

https://github.com/elafildecolor/To-Do-List/blob/8cec725ddb451b0a1b6095b55897a2fdb59754f4/modules/TaskRepository.js#L44-L47