Skip to content

Birmingham | ITP-Jan 2026 | Arunkumar Akilan | Sprint 2 | Book Library#417

Closed
arunkumarakilan wants to merge 4 commits intoCodeYourFuture:mainfrom
arunkumarakilan:booklibrary/sprint2
Closed

Birmingham | ITP-Jan 2026 | Arunkumar Akilan | Sprint 2 | Book Library#417
arunkumarakilan wants to merge 4 commits intoCodeYourFuture:mainfrom
arunkumarakilan:booklibrary/sprint2

Conversation

@arunkumarakilan
Copy link
Copy Markdown

🔧 What does this PR do?

This PR implements a book library application using JavaScript and DOM manipulation.

Features:

  • View a list of books
  • Add new books with title, author, pages, and read status
  • Validate inputs before adding books
  • Toggle read/unread status
  • Delete books from the list

🐞 Bugs Fixed

  • Fixed issue where books were not displaying on page load
  • Fixed error when adding a book
  • Fixed bug where title was used as author
  • Fixed delete button functionality
  • Fixed incorrect read status saving

🧪 How to test

  1. Open the app in browser
  2. Check initial books are displayed
  3. Add a new book
  4. Try submitting empty form → should show alert
  5. Toggle read status → should update
  6. Delete a book → should remove from list

✅ Self checklist

  • I have titled my PR correctly
  • My changes meet the requirements
  • I have tested my changes
  • My code follows the style guide

@github-actions
Copy link
Copy Markdown

The changed files in this PR don't match what is expected for this task.

Please check that you committed the right files for the task, and that there are no accidentally committed files from other sprints.

Please review the changed files tab at the top of the page, we are only expecting changes in this directory: ^debugging/book-library/

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above.

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