Self Access Issue

• Great on implementing OOP and DRY principle already with following KISS
• The Task List class commented code can be removed for clean coding . Click here to check
• The UI class can use map here, instead of forEach for clean code

Peer-to-peer code review

Good job so far, however, there is a small issue in your code that should be addressed.

• Kindly remove the commented lines of code in the tasklist.js file from lines 16 - 18. Commented lines of code will make your codebase look messy.

Good job so far 👍

• Clean code

Issues found

• It will be nice if you add strike-through on a todo when a task is mark completed
• Kindly check the live demo link in your REAME.md file, it isn't working.

Test Updates of tasks, and Clear Completed Status

• It's a pair programming task.
• Set up the project with jest
• Set up babel package and configuration to use ES6 as CommonJS by the jest
• Followed AAA pattern.
• Mocked localStorage from the super(parent) class as an empty array before each test initialization inside describe scope.
• Implemented fake mocking of external services from parent class.
• Done mock implementation of 2 functions - 1. setList and 2. reOrderTaskIndex of TaskList class (Parent Class) used by the unit testing class CRUD (Child Class) & CompletedTask (Completed Task Class).
• Added tests for editing the task description, updating an item's 'completed' status, and Clear all completed with reordering list and save to local storage.
• Used localStorage functionality from part 1. Re-order index implemented in this test.
• All tests for different classes are grouped with describe function respectively.

Test add and delete item functionality

• It's a pair programming task.
• Set up the project with jest
• Set up babel package and configuration to use ES6 as CommonJS by the jest
• Followed AAA pattern.
• Mocked localStorage from the super(parent) class as an empty array before each test initialization inside describe scope.
• Implemented fake mocking of external services from parent class.
• Done mock implementation of 2 functions - 1. setList and 2. reOrderTaskIndex of TaskList class (Parent Class) used by the unit testing class CRUD (Child Class).
• Added tests to add / remove exactly 1 item from the mock implemented local storage list while the tests run, and checked it using the comparison between updated list size and previous list size.

Added Review Changes: review 1

• Fixed error in Task index reordering.

Develop a simple interactive to-do list

• To see a live demo from this branch click here
• Used ES6 syntax
• Implemented ES6 modular structure using OOP - this way the application will be robust and scalable
• Implemented DRY and SOLID principles - this way code will be more maintainable in future
• The todo list can add, delete and edit tasks
• Created methods related to the status updates (completed: true / false).
• Added event listener to the checkbox to mark tasks completed
• Implemented a function for the "Clear all completed" button using filter() method to get only completed tasks
• Reorder index property of task object after removing completed tasks
• Implemented proper error handling if empty input been provide
• All todo list data are saved in localStorage
• Used webpack to serve the application from a bundle.

Develop a simple to-do list

• Used ES6 syntax

• Implemented ES6 modular structure using OOP - this way the application will be robust and scalable

• Implemented DRY and SOLID principles - this way code will be more maintainable in future

• The todo list can add, delete and edit tasks

• Task add can happen in 2 ways:

   1. Press Enter key after typing the task in the input box.
   2. After typing the task press the "+" button.
  

• If Add Task input field is empty - shows an error

• While editing a task if the description field is made empty alerts error

• Edit occurs on change event of the description

• All todo list data are saved in localStorage

• After the task is deleted index of the tasks gets adjusted in proper order before rendering

• Used webpack to serve the application from a bundle.

Review Changes: review 1

• As the reviewer suggested, refactored the code to dynamically build the task list li elements from the static array of the task objects.
• Commented out OOP related code for the time being, as these codes will be helpful in coming tasks for the same project.

Set up Todo List Skeleton with Webpack

• It's a solo Task.
• Used webpack to bundle JavaScript, HTML, and CSS.
• Implemented OOP with DRY and SOLID principals.
• Used ES6 modules to write modular JavaScript.
• Created Task object with three keys.
• On page load show task list dynamically from TaskList module
• Implemented all basic file structure index.html, index.js, and style.css