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

fix: track all index in the manager #660

Merged
merged 2 commits into from
Dec 1, 2017
Merged

Conversation

samouss
Copy link
Collaborator

@samouss samouss commented Nov 27, 2017

Summary

Fix #636

We already track the index for the shared parameters, but not for the main parameters. This PR aims to fix this issue by tracking all the indices.

Result

You can play with the example on Storybook.

@samouss samouss requested review from vvo and mthuret November 27, 2017 16:38
@samouss samouss changed the title fix: track all index fix: track all index in the manager Nov 27, 2017
@samouss samouss force-pushed the fix/track-all-index-parameters branch from f72bb7c to dd6a0f6 Compare November 27, 2017 16:39
@algobot
Copy link
Contributor

algobot commented Nov 27, 2017

Deploy preview ready!

Built with commit 663504d

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

@algolia algolia deleted a comment from algobot Nov 27, 2017
@samouss samouss requested a review from marielaures November 27, 2017 17:06
@mthuret
Copy link
Contributor

mthuret commented Nov 27, 2017

LGTM!
I'm wondering if we shouldn't introduce sublevel into the storybook. It's convenient to add stories to demonstrate those tricky use cases, but imo they do not have the same importance as stories such as "Autocomplete" here. Wdyt?

@samouss
Copy link
Collaborator Author

samouss commented Nov 27, 2017

Yep why not! Because this story is more related to a test case rather than documentation. It makes sense to me.

@samouss samouss requested a review from Haroenv November 28, 2017 09:04
@samouss samouss force-pushed the fix/track-all-index-parameters branch 2 times, most recently from 0ceb3b0 to 50a3dce Compare November 29, 2017 18:21
@samouss samouss force-pushed the fix/track-all-index-parameters branch from 50a3dce to 663504d Compare December 1, 2017 09:05
@samouss samouss merged commit 793502b into master Dec 1, 2017
@samouss samouss deleted the fix/track-all-index-parameters branch December 1, 2017 09:22
samouss added a commit that referenced this pull request Dec 20, 2017
<a name="4.3.0"></a>

* reset page with multi index ([#665](#665)) ([865b7dc](865b7dc))
* **SearchBox:** use hidden over style to hide loader ([#714](#714)) ([ed5eb77](ed5eb77))
* test recipes ([#740](#740)) ([de2cc37](de2cc37))
* track all index in the manager ([#660](#660)) ([793502b](793502b))

* loading indicator ([#544](#544)) ([189659e](189659e))
* **Highlight:** support array of strings ([#715](#715)) ([8e93c6a](8e93c6a))
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.

In a multi-index configuration, SortBy doesn't sort the main index once inside an Index component
3 participants