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

fix(connectRange): update default refinement propTypes #978

Merged
merged 2 commits into from
Feb 16, 2018

Conversation

samouss
Copy link
Collaborator

@samouss samouss commented Feb 15, 2018

Summary

Fix #564

With the current implementation when we provide a defaultRefinement to connectRange we must pass an object with both min & max. There are some cases when we want to pass only a min, max or even an empty object. In the latter case it allow the widget to be "controlled" by the defaultRefinement. See #892.

@samouss samouss requested review from Haroenv and vvo February 15, 2018 18:18
@samouss samouss force-pushed the fix/connect-range-default-refinement branch from 26209f3 to ee99dad Compare February 15, 2018 18:19
@samouss samouss changed the title fix(connectRange): update default refinement types fix(connectRange): update default refinement propTypes Feb 15, 2018
@algobot
Copy link
Contributor

algobot commented Feb 15, 2018

Deploy preview for react-instantsearch ready!

Built with commit e1304c0

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

@algolia algolia deleted a comment from algobot Feb 15, 2018
@samouss samouss force-pushed the fix/connect-range-default-refinement branch from ee99dad to 0908610 Compare February 16, 2018 08:30
@samouss samouss force-pushed the fix/connect-range-default-refinement branch from 0908610 to e1304c0 Compare February 16, 2018 09:10
@samouss samouss merged commit c065fb1 into master Feb 16, 2018
@samouss samouss deleted the fix/connect-range-default-refinement branch February 16, 2018 13:15
samouss added a commit that referenced this pull request Mar 6, 2018
<a name="4.5.2"></a>
## [4.5.2](v4.5.1...v4.5.2) (2018-03-06)

### Bug Fixes

* **connectRange:** update default refinement propTypes ([#978](#978)) ([c065fb1](c065fb1))
* **IndexUtils:** avoid throw an error when cleanUp multi index ([#1019](#1019)) ([865a3c3](865a3c3))
* **SearchBox:** avoid to bind click on reset button ([#979](#979)) ([ea3063a](ea3063a))
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.

None yet

3 participants