Skip to content

London | 26-ITP-May | Dipa Sarker | Sprint 3 | ToDo List App - #1378

Open
Dipa-Sarker wants to merge 1 commit into
CodeYourFuture:mainfrom
Dipa-Sarker:Sprint-3/To-do-list-App
Open

London | 26-ITP-May | Dipa Sarker | Sprint 3 | ToDo List App#1378
Dipa-Sarker wants to merge 1 commit into
CodeYourFuture:mainfrom
Dipa-Sarker:Sprint-3/To-do-list-App

Conversation

@Dipa-Sarker

Copy link
Copy Markdown

Learners, PR Template

Self checklist

  • I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
  • My changes meet the requirements of the task
  • I have tested my changes
  • My changes follow the style guide

Changelist

  • Changes Made
  • Added a "Delete Completed Tasks" button to index.html.
  • Created a new deleteCompleted(todos) function in todos.mjs to remove all completed tasks from the todos array.
  • Added Jest tests in todos.test.mjs to verify that completed tasks are deleted correctly and incomplete tasks remain.
  • Added a deleteCompletedTasks() handler in script.js.
  • Connected the new button to the handler using an event listener.
  • Re-render the ToDo list after deleting completed tasks so the UI stays in sync with the data.

-Testing

  • Ran the existing test suite and confirmed all tests pass.

  • Verified in the browser that:

    • Completed tasks are removed when the button is clicked.
    • Incomplete tasks remain in the list.
    • Existing add, complete, and delete functionality continues to work.

@Dipa-Sarker Dipa-Sarker added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Aug 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant