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

feat: custom root index #792

Merged
merged 5 commits into from
Jan 4, 2018
Merged

feat: custom root index #792

merged 5 commits into from
Jan 4, 2018

Conversation

samouss
Copy link
Collaborator

@samouss samouss commented Jan 3, 2018

Summary

In a previous PR #770 we have introduce the root prop for the InstantSearch component. But we forgot to handle the case for the Index component. The test of createInstantSearch has also be updated because the previous test didn't tested the root prop.

Enable the possibility to pass custom root prop to Index. Don't introduce any breaking change, since we use the root of createIndex as defaultProps.

Result

You can play with the example in Storybook.

@samouss samouss requested review from Haroenv and mthuret January 3, 2018 17:23
@samouss samouss changed the title Fix/custom root index feat: custom root index Jan 3, 2018
@Haroenv
Copy link
Contributor

Haroenv commented Jan 3, 2018

looks good when skimming

@algobot
Copy link
Contributor

algobot commented Jan 3, 2018

Deploy preview ready!

Built with commit 1cc2acb

https://deploy-preview-792--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.

Awesome LGTM! let's merge this and I'll fix my QS demo.

@samouss samouss force-pushed the fix/custom-root-index branch from 9d58256 to 1cc2acb Compare January 4, 2018 16:03
@samouss samouss merged commit d793b0a into master Jan 4, 2018
@samouss samouss deleted the fix/custom-root-index branch January 4, 2018 16:47
samouss added a commit that referenced this pull request Jan 9, 2018
<a name="4.4.1"></a>
## [4.4.1](v4.4.0...v4.4.1) (2018-01-09)

### Bug Fixes

* **SearchBox**: clear SearchBox without search as you type ([#802](#802)) ([c49b2b6](c49b2b6))
* **connectRange:** check if facet exist before access ([#797](#797)) ([6520760](6520760))
* **stories:** avoid to use linear-background it breaks Argos every time ([#804](#804)) ([0beded7](0beded7))
* **stories:** limit hits per page on Index ([#806](#806)) ([6eb14d3](6eb14d3))

### Features

* **Index:** allow custom root ([#792](#792)) ([d793b0a](d793b0a))
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.

4 participants