Skip to content

London | 26-ITP-January | Eugenie Ahangama | Sprint 3 | Todo List App#1006

Open
Eugenie-A wants to merge 4 commits intoCodeYourFuture:mainfrom
Eugenie-A:Sprint-3/todo-list
Open

London | 26-ITP-January | Eugenie Ahangama | Sprint 3 | Todo List App#1006
Eugenie-A wants to merge 4 commits intoCodeYourFuture:mainfrom
Eugenie-A:Sprint-3/todo-list

Conversation

@Eugenie-A
Copy link

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

  • Added the "Delete completed tasks" button in index.html
  • Implemented deleteCompleted() in todos.mjs
  • Wrote Jest tests for deleteCompleted() in todos.test.mjs
  • Added event listener for delete completed button in script.mjs
  • Implemented deleteCompletedTodos callback in script.mjs

Questions

Why is render without parentheses a bug?

@Eugenie-A Eugenie-A added 🏕 Priority Mandatory This work is expected 🐂 Size Medium 1-4 hours 📅 Sprint 3 Assigned during Sprint 3 of this module Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. Submit:PR Module-Data-Groups The name of the module. Core This is a core task and should be completed by all trainees labels Mar 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Core This is a core task and should be completed by all trainees Module-Data-Groups The name of the module. Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. 🏕 Priority Mandatory This work is expected 🐂 Size Medium 1-4 hours 📅 Sprint 3 Assigned during Sprint 3 of this module Submit:PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant