Morning session: Peer-to-peer code review Module 2 - Week 3 - 22 Feb 2022

[Tufahel]

Hi @Elerqsousy, You have done a great job. Every functionality of the project is ok and the interface is also looking good. Here are some suggestions that you can consider :
1.

• You can consider using retrun this inside the class for method chaining. Here is an example from this article :
  
  

• Inside this function, it exists a single line. This is not a multiline function and it's a good practice for writing javascript code not to use multiline for single line function.
  

  to-do-list/src/modules/functionality.js

  Line 50 in 6a83d1c

  newList.forEach((e) => {

• This to-do list is taking blank comments. Consider using required inside input tag.
  

Happy Coding 👍

[damdafayton]

• https://github.com/Elerqsousy/to-do-list/blob/main/src/modules/object-class.js#L5
  line not necessary

• https://github.com/Elerqsousy/to-do-list/blob/main/src/modules/functionality.js
  Class for data is not necessary since there is only one data object for each app