Skip to content

Conversation

@Adil-Akothiat
Copy link
Contributor

@Adil-Akothiat Adil-Akothiat commented Dec 19, 2025

I Implmented searchBooks and filterBooks features, I created readme.md that describe the challenge(documentation), I add new property for the Library class called totalBooks that count the total number of books in the library, the property is incremented by 1 when we create a new book.

Pull Request Type

What kind of change does this PR introduce?

  • Solved an issue/task
  • Suggest new feature/task for Fork, Commit, Merge
  • Other... Please describe:

Issue Number: N/A

Issue Number: [Insert Issue Number (if applicable)]

Info For Contributors

Remember NOT to close the Pull Request yourself, wait until it gets merged or changes are requested.

If you are asked to make changes to your code, you can do so by committing them to the same branch. There is no need to close the current Pull Request and open a new one.

@Adil-Akothiat Adil-Akothiat changed the title implment searchBooks and filterBooks features + add readme.md that describe the challenge details Implement book search and filter features and add challenge documentation for JavaScript task(hard) Dec 19, 2025
@nikohoffren nikohoffren merged commit f646664 into fork-commit-merge:main Dec 20, 2025
4 checks passed
@nikohoffren
Copy link
Member

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your Pull Request has been merged, and the files have been reset for the next contributor.

What's next?

If you're looking for more ways to contribute, we invite you to check out our forkcommitmerge.dev website repository. The repository contains real issues you can help resolve, or you can create some of your own if you notice any bugs or want to suggest new features. You can also check out the Influences section in the README to find more projects similar to this one.

Also please leave a star to this project if you found it helpful.

We look forward to seeing your contributions!

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.

2 participants