Cape Town | 2026-ITP-Jan | Isaac Abodunrin | Sprint 2 | Data Flows: Book Library Project

[bytesandroses]

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

Fixed bugs and syntax errors in the script.js file, ensuring:

1. Books are displayed when page loads
2. New books are added correctly
3. Author name is correctly saved
4. Deletion now removes books
5. 'Read' option correctly processed

[cjyuan]

1. If your PR is ready to be reviewed, you need to add a "Needs Review" label.

2. Regardless, please check if any of this general feedback can help you further improve your code? https://github.com/CodeYourFuture/Module-Data-Flows/blob/general-review-feedback/debugging/book-library/feedback.md

Doing so can help me speed up the review process. Thanks.

[bytesandroses]

Hi @cjyuan sorry I forgot to add the needs feed back label.

I've gone ahead .and refactored the script meet the criteria described on the feedback link

[cjyuan]

Code looks good. Well done.

[cjyuan]

Common practice is to declare all shared variables/constants at the beginning of the file before function definition.

[cjyuan]

Could consider keeping all the "run on page load" code in one place (in the call back of window's onload event listener) as:

const submitForm = document.getElementById("bookForm"); // declare at the top
...

// function definition
function submit(e) {
  ...
}

window.addEventListener('load', () => {
  submitForm.addEventListener("submit", submit);
  populateStorage();
  render();
});

[illicitonion]

Closing PR because the January ITP run has finished. Feel free to re-open if you're still working on it.