Hello @joseph07-drack 👋 you did Amazing work 🥇 .

• Clean Code 👍 .
• Github workflow 👍 .

but I noticed that the live demo link is not working, there is a typo on the URL.

1. kindly fix that.

Learning objective is to understand how to:

    - use webpack to bundle JavaScript.
    - learn how to use proper ES6 syntax.
    - use ES6 modules to write modular JavaScript. 

• Used jest framework for testing.
• Created a test file(s) ([..].test.js) for a file(s) containing the following functions:
• a function for editing the task description.
• a function for updating an item's 'completed' status.
• the "Clear all completed" function.
• Used the mock storage object you have created in Part 1 of this project to mock the localStorage updates.
• Mocked the HTML elements to test DOM manipulation functions.
• Made sure you group your tests using the describe() method.

JEST TESTING REVIEW

First, I will say I like the way, you kept the code DRY and your use of desctructuring. Altogther, you have a great project 👍 . But I feel like you can use some better practices to deliver KISS and DRY codes. Below are the issues:

• [ ] The null input was not taken care of in the add method.
• [ ] Kindly try to keep it simple with remove testing code.

Learning Objectives:

Write unit tests for a JavaScript app.

Required Changes Implemented:

• This is a pair-programming project. Please decide whose project will you use for testing.
• Use jest framework for testing.
• Create a test file ([..].test.js) for a file containing the add item and delete item functions that you want to test.
• Take a look at each of these functions. Are they pure functions? If the answer is "yes" then writing tests for them should be straightforward. Some of those functions however will update localStorage and manipulate the DOM. For those, you will need to use mocks:
• Mock a storage object to "imitate" localStorage operations (only if you are using jest < 24.x, in newer versions localStorage is mocked automatically).
• Mock HTML to test if add/delete functions add or remove exactly one
• element to/from the list in the DOM.
• Make sure you group your tests using the describe() method.

Changes:

• Now, when adding, removing, updating something in the localstorage, no reloads
• Increase the usage of DRY, KISS and YAGN approaches
• Less line of codes to get the feature done

Code quality checking (Javascript Best practices, DRY, KISS and YAGNI)

• [ ] JavaScript Best Practices: - window.location.reload() is not a good practice, find a way to include the event.preventDefault() so that the page may not reload every time and an action get trigged. check (actions.js line 10 and line 27 and ./src/index.js line 72) - unused codes used not be part of the codebase, kindly remove the ./modules/todos.js file. - avoid using verbose code for code readability and maitenance. Kindly check this and refactor.

• [ ] DRY - I noticed there is localstorage snippet to store items that is repeated more than one time : (actions.js line 8, line 26 and line 32)

• [ ] KISS Everything is simple 👍

• [ ] YAGNI No unnecessary code complexy 👍