Skip to content

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

Open
arunkumarakilan wants to merge 1 commit intoCodeYourFuture:mainfrom
arunkumarakilan:booklibrary/final-clean
Open

Birmingham | ITP-Jan 2026 | Arunkumar Akilan | Sprint 2 | Book Library#422
arunkumarakilan wants to merge 1 commit intoCodeYourFuture:mainfrom
arunkumarakilan:booklibrary/final-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 incorrectly used as author
  • Fixed delete button functionality
  • Fixed incorrect read status saving

🧪 How to test

  1. Open the app in the browser
  2. Verify default books are displayed
  3. Add a new book with all fields filled
  4. Try submitting empty fields → should show alert
  5. Toggle read/unread status using the button
  6. Delete a book using the delete button

📁 Files included

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

✅ Self checklist

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

@arunkumarakilan arunkumarakilan added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Apr 11, 2026
@Luro91 Luro91 added Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. and removed Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Apr 12, 2026
Copy link
Copy Markdown

@Luro91 Luro91 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very well done. The app works now correctly

@Luro91 Luro91 added Complete Volunteer to add when work is complete and all review comments have been addressed. and removed Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. labels Apr 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Complete Volunteer to add when work is complete and all review comments have been addressed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants