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

fix(connectRange): use refine instead of cleanUp in metadata #526

Merged
merged 2 commits into from
Oct 24, 2017

Conversation

samouss
Copy link
Collaborator

@samouss samouss commented Oct 23, 2017

Summary

Fix #504

This PR replace the call to cleanUp for refine in getMetadata. The others connectors implementations always use refine for clear the value from getMetadata (ex: connectRefinementList).

Result

You can play with the RangeInput & RangeSlider on Storybook.

Before:

before

After:

after

@algobot
Copy link
Contributor

algobot commented Oct 23, 2017

Deploy preview ready!

Built with commit 662f3da

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

Copy link
Contributor

@mthuret mthuret 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/connect-range-default-refinement branch from 10ab2b6 to 28f2ec0 Compare October 23, 2017 15:36
@mthuret mthuret merged commit 1861235 into master Oct 24, 2017
@mthuret mthuret deleted the fix/connect-range-default-refinement branch October 24, 2017 08:38
samouss added a commit that referenced this pull request Nov 2, 2017
<a name="4.2.0"></a>
# [4.2.0](v4.1.3...v4.2.0) (2017-11-02)

### Bug Fixes

* **connectRange:** handle boundaries on first call ([9f14dc0](9f14dc0))
* **connectRange:** use refine instead of cleanUp in metadata ([#526](#526)) ([1861235](1861235))
* **hierarchicaMenu:** allow sorting and using limit ([fe178ed](fe178ed)), closes [#92](#92)
* **InfiniteHits:** add disabled style to the LoadMore button ([#477](#477)) ([faba1ad](faba1ad))
* **Range:** handle float, allow reset and respect boundaries ([75969b8](75969b8))
* **RangeInput:** fix compatibility with React 16 & Panel ([3f218db](3f218db))
* **searchbox:** add maxlength 512 ([#542](#542)) ([5bd4033](5bd4033)), closes [#510](#510)

### Features

* **MenuSelect:** add component and connector ([cc6e0d7](cc6e0d7))
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