Clear all

[JohnFTitor]

Hi there 👋🏻

Thanks for taking the time to review the project. Here are some things important to note about it:

Sass

The project was built using Sass. The only difference was that I needed to add an additional loader to webpack, but the result is overall the same (A CSS output). Additionally, I used MiniCssExtractPlugin to generate an external CSS file in the dist folder instead of the styles in the head of the document. It has the same dynamic response, it's just that the output looks more organized.

Now into the project...

Changes implemented in "clearAll" branch

• ✅ My code follows the style guidelines of this project.
• ✅ Create functions to set the status of a task object and to filter completed items from an array.
• ✅ Add event listener to checkbox to update completed status of its task object.
• ✅ Filter completed items and remove each of them using the same functions defined before.

Final notes

Is expected that this project meets the requirements. Please let me know if anything is left to be done or can be improved. Again, thanks for the review.

[ShoiraTa]

STATUS: APPROVED ✔️

---

Hi @JohnFTitor ,

Your project is complete! Great job for nailing it right on the first attempt 🥇. There is nothing else to say other than... it's time to merge it 🍾🚢 . Congratulations! 🎉💯🌟

Good Points 👍

• Project is set up correctly ✔️
• Descriptive Pull Request ✔️
• The task object's values for completed is updated upon user actions. ✔️
• A function for the "Clear all completed" button using the Array.filter() ✔️
• The updated array of items is stored in local storage, so the user gets the correct list values after the page reloads. ✔️

Cheers..Happy coding!..and keep soaring higher! 💻 🍷 🚀

---

As described in the Code reviews limits policy you have 2 more limited reviews per this project. If you think that the code review was not fair, you can request a second opinion using this form.