Skip to content

Brussels | ITP-2026-1 | Meysam Razagh | Debugging | Book library#93

Open
Epunch wants to merge 8 commits into
HackYourFutureBelgium:mainfrom
Epunch:book-library
Open

Brussels | ITP-2026-1 | Meysam Razagh | Debugging | Book library#93
Epunch wants to merge 8 commits into
HackYourFutureBelgium:mainfrom
Epunch:book-library

Conversation

@Epunch
Copy link
Copy Markdown

@Epunch Epunch commented May 20, 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

Description

I have successfully debugged and fixed all the major and minor issues in the Book Library project.

Bugs Fixed

  • Syntax Error: Fixed the broken for loop syntax in the render function that was crashing the script.
  • Delete Button: Fixed the variable name mismatch (delBut vs delButton) and changed the event listener from clicks to click.
  • Submit Error: Corrected the array reference from library to myLibrary inside the submit function.
  • Author Bug: Fixed the form constructor mapping so it correctly saves the author's name instead of duplicating the book title.
  • Read Status Logic: Corrected the inverted if condition so the correct "Yes/No" status displays on the screen.

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.

1 participant