Skip to content

London | 26-ITP-January | Eugenie Ahangama | Sprint 2 | Book Library#434

Open
Eugenie-A wants to merge 2 commits intoCodeYourFuture:mainfrom
Eugenie-A:book-library
Open

London | 26-ITP-January | Eugenie Ahangama | Sprint 2 | Book Library#434
Eugenie-A wants to merge 2 commits intoCodeYourFuture:mainfrom
Eugenie-A:book-library

Conversation

@Eugenie-A
Copy link
Copy Markdown

@Eugenie-A Eugenie-A commented Apr 14, 2026

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

index.html:

  • Fixed empty page title

script.js:

  • Fixed syntax error in for loop that prevented any books from displaying on load
  • Fixed library reference to myLibrary so new books are added correctly
  • Fixed author field using title value instead of author value
  • Fixed delete button variable name mismatch and wrong event name "clicks" to "click"
  • Fixed inverted read status logic so "Yes" and "No" display correctly
  • Added author field validation so books can't be added without an author

Questions

N/A

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

Reviewers

No reviews

Assignees

No one assigned

Labels

Core This is a core task and should be completed by all trainees Module-Data-Flows 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 2 Assigned during Sprint 2 of this module Submit:PR

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Eugenie-A