London|ITP-Jan-2026 |Alexandru Pocovnicu | Sprint 2 | Book library #410

Open

alexandru-pocovnicu wants to merge 17 commits intoCodeYourFuture:mainfrom

alexandru-pocovnicu:book-library

alexandru-pocovnicu commented Apr 6, 2026 •

edited

Loading

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

debugged the code

alexandru-pocovnicu added 10 commits

April 6, 2026 20:31


          Fix: Corrected variable name from 'library' to 'myLibrary' in submit …

026ab7c

…function


          Fix: Corrected syntax error in the render function's for loop

ba1e8fc


          Fix: Corrected button creation and event listener for delete function…

…ality in render function


          Fix: Corrected validation check for author input in submit function

2be9382


          Fix: Corrected logic for book read status in submit function

2ce8154


          Fix: Corrected event listener type for delete button in render function

e160341


          Fix: Corrected author value assignment in book creation within submit…

62672c3

… function


          Fix: Corrected book creation to use checked state for read status in …

31236b1

…submit function


          Fix: Corrected read status display logic in render function

88c944d


          Fix: Added validation for pages input in submit function

e9a419f

This comment has been minimized.

Sign in to view

4 similar comments

This comment has been minimized.

Sign in to view

This comment has been minimized.

Sign in to view

This comment has been minimized.

Sign in to view

This comment has been minimized.

Sign in to view

alexandru-pocovnicu added the Needs Review label

This comment has been minimized.

Sign in to view

github-actions bot removed the Needs Review label

alexandru-pocovnicu changed the title ~~London|26-ITP-January|Alexandru Pocovnicu|Sprint-2|Book library~~ London|26-ITP-January|Alexandru Pocovnicu|Sprint 2|Book library

This comment has been minimized.

Sign in to view

alexandru-pocovnicu added the Needs Review label

This comment has been minimized.

Sign in to view

github-actions bot removed the Needs Review label

alexandru-pocovnicu changed the title ~~London|26-ITP-January|Alexandru Pocovnicu|Sprint 2|Book library~~ London|ITP-Jan-2026 |Alexandru Pocovnicu | Sprint 2 | Book library

alexandru-pocovnicu added the Needs Review label

This comment has been minimized.

Sign in to view

github-actions bot removed the Needs Review label

This comment has been minimized.

Sign in to view

alexandru-pocovnicu added Needs Review labels

alexandru-pocovnicu added 4 commits

April 6, 2026 21:30


          Fix: Added language attribute to html tag and updated title in index.…

9e5e015

…html


          Fix: Standardized HTML syntax and formatting in index.html

6eef9d9


          delete trailing slash

cc880b1


          Fix: Corrected input types for title and author fields in index.html

0a0dcf3

cjyuan reviewed

View reviewed changes

Contributor

cjyuan left a comment

Can you check if any of this general feedback can help you further improve your code?
https://github.com/cjyuan/Module-Data-Flows/blob/book-library-feedback/debugging/book-library/feedback.md

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

cjyuan added Reviewed and removed Needs Review labels

alexandru-pocovnicu added 3 commits

April 9, 2026 15:11


          Fix: Corrected meta tag and input element syntax in index.html

8e1e374


          Fix: Removed unnecessary align-self property from .form-group in styl…

02d14fc

…e.css


          Fix: Corrected self-closing tags and standardized HTML syntax in inde…

ca4167d

…x.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels