Cape Town | 26-ITP-May | Shafiek Walker | Sprint 2 | Book Library

[shafiekwalker7861]

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

Fixed the Book Library application bugs.

• Fixed syntax error preventing books from loading
• Fixed the incorrect library array reference when adding books
• Fixed author input so the correct author is saved
• Added validation for missing title, author and page information
• Fixed read/unread status
• Fixed read status toggle
• Fixed delete button functionality
• Fixed incorrect delete event listener
• Cleared the form after successfully adding a book

Testing

Tested:

• Initial books load correctly
• New books can be added
• Missing fields are rejected
• Read status is saved correctly
• Read/unread status can be toggled
• Books can be deleted
• No console errors

[cjyuan]

Coud you 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.

[shafiekwalker7861]

Hi @cjyuan , thank you for the feedback. I reviewed the general feedback and made further improvements to the Book Library, including HTML validation, cleaner DOM variable names, numeric page counts, module-based script loading, improved table rendering, and non-blocking delete feedback. The changes have been pushed to the PR for review.

[cjyuan]

Looks good. Well done.

[shafiekwalker7861]

Thanks, CJ

…

On Tue, Aug 18, 2026 at 2:43 PM CJ Yuan ***@***.***> wrote: ***@***.**** commented on this pull request. Looks good. Well done. — Reply to this email directly, view it on GitHub <#566?email_source=notifications&email_token=A4CDQ6VBFQPN6C6CPO4U26T5KRFO7A5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTIOJWGEYTANRUGQZ2M4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2KYZTPN52GK4S7MNWGSY3L#pullrequestreview-4961106443>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/A4CDQ6RCL7QYCLHODJLG4TL5KRFO7AVCNFSNUABFKJSXA33TNF2G64TZHM3TSOJSGQ2TQMRXHNEXG43VMU5TKMJYGA4TKMZRHA4KC5QC> . Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS <https://github.com/notifications/mobile/ios/A4CDQ6TU7AMG5YHFNAZNIHD5KRFO7A5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTIOJWGEYTANRUGQZ2M4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2KUZTPN52GK4S7NFXXG> and Android <https://github.com/notifications/mobile/android/A4CDQ6VYXF6LCSBW7NWPMLD5KRFO7A5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTIOJWGEYTANRUGQZ2M4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2K4ZTPN52GK4S7MFXGI4TPNFSA>. Download it today! You are receiving this because you authored the thread.Message ID: ***@***.***>