Skip to content
This repository has been archived by the owner on Dec 30, 2022. It is now read-only.

fix: pass maxFacetHits to SFFV #863

Merged
merged 7 commits into from
Jan 20, 2018
Merged

fix: pass maxFacetHits to SFFV #863

merged 7 commits into from
Jan 20, 2018

Conversation

samouss
Copy link
Collaborator

@samouss samouss commented Jan 17, 2018

Summary

Fix #707

Inside the RefinementList & Menu we can search through the list of items. On the first render the maxValuesPerFacet is correctly set on the SearchParameters. But when you trigger a search from the SearchBox the limit provided to the widget is ignored and only the default one is applied (which is 10). It can produce unexpected results like the one linked inside the issue. The limitMax is set to 30 but when you search for some facets since the latter is ignored you only have 10 results which is equal to the limitMin and so the button is disabled.

In the following example I search for <space> so all the results are retrieve until the limit (15).

Before:

before

After:

after

@samouss samouss requested review from vvo, Haroenv and mthuret January 17, 2018 12:01
@samouss samouss force-pushed the fix/show-more-limit branch from c25eb46 to da13a90 Compare January 17, 2018 12:02
@algolia algolia deleted a comment from algobot Jan 17, 2018
@algobot
Copy link
Contributor

algobot commented Jan 17, 2018

Deploy preview for react-instantsearch ready!

Built with commit 54e348b

https://deploy-preview-863--react-instantsearch.netlify.com

@samouss samouss force-pushed the fix/show-more-limit branch 4 times, most recently from aec6b62 to 0c2c0ca Compare January 19, 2018 10:44
Copy link
Contributor

@vvo vvo left a comment

Choose a reason for hiding this comment

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

LGTM

@samouss samouss force-pushed the fix/show-more-limit branch from 0c2c0ca to 54e348b Compare January 20, 2018 13:04
@samouss samouss merged commit de23a46 into master Jan 20, 2018
@samouss samouss deleted the fix/show-more-limit branch January 20, 2018 13:39
samouss added a commit that referenced this pull request Jan 24, 2018
<a name="4.4.2"></a>
## [4.4.2](v4.4.1...v4.4.2) (2018-01-24)

### Bug Fixes

* **currentRefinements:** give access to id and index from transformItems for deduplication ([#830](#830)) ([316b8f5](316b8f5))
* pass maxFacetHits to SFFV ([#863](#863)) ([de23a46](de23a46))
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants