search

od-c0d3r / TodoListJs

Html-Css-Js simple app
1 stars 0 forks source link

Merge `Interactive` into `main` branch #2

Closed od-c0d3r closed 2 years ago

od-c0d3r commented 2 years ago

What's new :question:

  1. User can change the task completed statues from true <=> false
  2. Update README file
  3. The list array is begin hosted in localStorage API
mimipeshy commented 2 years ago

Hello @od-c0d3r 👋,

This is an additional review as a result of the second opinion request.

From your project, I am able to mark a complete todo as true but once I reload the page it defaults back to false. So this means that you should persist your data according to the project requirements shown here ⬇️ image

I hope this is clear enough and will unblock you. Kindly request for another review when done 😄

Keep rocking 🚀!

od-c0d3r commented 2 years ago

Thanks for the review. i wasn't able to see the whole picture. Your clarification helped me @DeliceLydia, @mimipeshy