ITP-JAN-25 | LONDON | Bakytbek Sydykov | Module-Data-Flows | book-library

[Bahasyd]

Learners, PR Template

Self checklist

• I have committed my files one by one, on purpose, and for a reason
• I have titled my PR with COHORT_NAME | FIRST_NAME LAST_NAME | REPO_NAME | WEEK
• I have tested my changes
• My changes follow the style guide
• My changes meet the requirements of this task

Changelist

Explain my PR: Cleared form fields after successfully adding a book to the library.
The latest changes — I added the clearing of this line after the search.

Questions

Ask any questions you have for your reviewer.

[jenny-alexander]

1. Please remove any comments that are not required. This will make your code cleaner, easier to read and easier to maintain in the future.

2. Currently your application does not have a title in the browser tab (see screenshot). Can you update the application title to something more meaningful?

3. After adding a new book to the library, the form still contains the books details. If I want to add another book, I have to clear out the form fields. Can you update the application to clear out the fields after successfully adding a book to the library?

4. Please make sure to update the Changelist section of the PR with a short explanation of your changes.

[Bahasyd]

Cleared form fields after successfully adding a book to the library.

[cjyuan]

@Bahasyd Can you give a brief description to this PR in the "Changelist" section in the first comment (as requested by the reviewer)

Please make sure to update the Changelist section of the PR with a short explanation of your changes.

[jenny-alexander]

If code is outside of a function, it will only run when the script first loads.
For this application, it makes sense for the book form to be cleared after successful submission.

Can you review this code change, try it out locally and make the changes necessary to clear the form after a submit?