1. Keep all the JavaScript code in JS files, not in the script tag. ✅

2. Keep your Code Clean following this advice:

   • type checks. ✅
   • naming. ✅
   • simplicity. ✅
   • DRY. ✅

3. If you use node modules, .lock files should be in the repository to avoid problems with future versions of the dependencies. ✅

4. If you use ES6, use object destructuring to get the values from an object. This way you can avoid repeating a lot of code. ⛔

5. Do not commit console.log to your repo. It’s ugly, it kills performance and it can make confidential data be visible to anyone using the browser tools to look at your website. ✅

6. Do not use window.alert() or window.confirm(). It’s ugly, impossible to style, stops code execution, and displays differently on different browsers. Use custom modal instead. ✅

7. Keep the number of changes/updates to the DOM as low as possible, they are very expensive for the browser. ✅

8. Keep the application logic separated from DOM manipulation tasks. ✅

9. Do not use document.write or eval ✅

10. Add node_modules dir to your .gitignore file as all those files are not needed in your repo (each team member will install all packages thanks to your package.json file). ✅

11. Do not commit old pieces of code as inline comments. They will make your project look messy. If you need to review a previous version of your code, you can always use git history. ✅

Great implementation. Regarding the specifications there is nothing to comment on.

Animations are nice, colors are pleasing and using Scss also was a good learning practice.

Great job.

In this Pull Request, We added the following unit tests:

• To add a new task. ✅
• To remove an existing task. ✅
• To check if a Task already exists. ✅

In the second part of this Pull Request, We implemented the following unit tests:

• To check if a task has been added correctly. ✅
• To check if a task status updates correctly. ✅
• To check if the clear all completed function works correctly. ✅

In this Pull Request, the following changes were implemented:

• Added methods related to the status updates. ✅
• Add event listener to the checkbox. ✅
• Update items object's value for completed key upon user actions. ✅
• Implement a function for the "Clear all completed" button. ✅
• Store the updated array of items in local storage, so the user gets the correct list values after the page reloads. ✅

Live Demo Link

In this Pull Request, the following changes were implemented:

• Removed all hardcoded items from the tasks array. ✅
• Created new JavaScript files for the new functionality. ✅
• Implemented a function for adding a new task. ✅
• Implemented a function for deleting a task. ✅
• Implemented a function for editing task descriptions. ✅
• By default new tasks have the property completed set to false and the property index set to the new array length. ✅
• Deleting a task updates all remaining items' indexes, so they represent the current list order and are unique. ✅
• All changes to the To-Do List are saved in local storage. ✅

Live Demo Link

In this Pull Request:

• Added HTML & CSS to style the basic To-Do List Structure ✅
• Added Tasks dynamically using JavaScript. ✅
• Used Webpack to Create bundle everything. ✅
• A deployed project to Github pages. ✅