Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix search filter for 'All' languages facet #21

Merged
merged 1 commit into from
Mar 5, 2024

Conversation

attemoi
Copy link
Contributor

@attemoi attemoi commented Mar 5, 2024

When selecting the 'All' option under language facet, the search was being performed only against languages defined for the library instead of all the languages in the collection.

Now it should will show books in any language that exists in the collection.

  • I have updated the documentation accordingly.
  • All new and existing tests passed.

@attemoi attemoi force-pushed the bugfix/simplye-209/all-languages branch 2 times, most recently from 60f5393 to b89c1cf Compare March 5, 2024 10:23
When selecting the 'All' option under language facet, the search
was being performed only against languages defined for the library
instead of all the languages in the collection.

Now it should will show books in any language that exists in the
collection.
@attemoi attemoi force-pushed the bugfix/simplye-209/all-languages branch from b89c1cf to 5ce251d Compare March 5, 2024 11:16
@attemoi attemoi marked this pull request as ready for review March 5, 2024 11:51
@attemoi attemoi merged commit fcfc228 into main Mar 5, 2024
19 checks passed
@attemoi attemoi deleted the bugfix/simplye-209/all-languages branch March 5, 2024 13:01
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