Skip to content

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

Open
arunkumarakilan wants to merge 4 commits intoCodeYourFuture:mainfrom
arunkumarakilan:booklibrary/clean
Open

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

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 on page load
  • Add new books with title, author, pages, and read status
  • Form validation to prevent empty inputs
  • Toggle read/unread status
  • Delete books from the list

🐞 Bugs Fixed

  • Fixed issue where books were not displayed on page load
  • Fixed console 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. Verify default books are displayed
  3. Add a new book with all fields
  4. Try submitting empty fields → should show alert
  5. Click "Yes/No" to toggle read status
  6. Click "Delete" to remove a book

📁 Note on files

This PR includes only the required book-library files:

  • debugging/book-library/index.html
  • debugging/book-library/script.js
  • debugging/book-library/style.css

Unrelated files have been removed to keep the PR clean and focused.


✅ 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