Skip to content

To Do list: interactive list#3

Open
evans22j wants to merge 12 commits intomainfrom
interactive-list
Open

To Do list: interactive list#3
evans22j wants to merge 12 commits intomainfrom
interactive-list

Conversation

@evans22j
Copy link
Owner

@evans22j evans22j commented Apr 2, 2022

In this milestone

  • Add a new JavaScript file and import it as a module
  • Add event listener to the checkbox (change).
  • Update items object's value for completed key upon user actions.
  • Implement a function for the "Clear all completed" button (use filter() method).
  • Store the updated array of items in local storage, so the user gets the correct list
    values after the page reloads.

@netlify
Copy link

netlify bot commented Apr 2, 2022

Deploy Preview for celebrated-otter-40acef ready!

Name Link
🔨 Latest commit 04aa02c
🔍 Latest deploy log https://app.netlify.com/sites/celebrated-otter-40acef/deploys/624870ad415ab10009082f9d
😎 Deploy Preview https://deploy-preview-3--celebrated-otter-40acef.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Copy link

@mmsesay mmsesay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @evans22j,

Your project is complete! There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Cheers and Happy coding!👏👏👏

Feel free to leave any questions or comments in the PR thread if something is not 100% clear.


As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants