Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(insights): option to disable automatic insights middleware #5493

Merged
merged 3 commits into from
Feb 23, 2023

Conversation

Haroenv
Copy link
Contributor

@Haroenv Haroenv commented Feb 21, 2023

instead of passing your own middleware, where timing could be tricky, to disable the internal middleware you pass insights: false to InstantSearch. Note that the internal middleware will still be removed when a different middleware is passed to allow easy customisation

FX-2250

@codesandbox-ci
Copy link

codesandbox-ci bot commented Feb 21, 2023

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit c76dd8e:

Sandbox Source
InstantSearch.js Configuration
react-instantsearch-app Configuration
example-react-instantsearch-hooks-default-theme Configuration
example-vue-instantsearch-default-theme Configuration

@Haroenv Haroenv changed the title feat(insights): option to enable feat(insights): option to disable Feb 21, 2023
@Haroenv Haroenv marked this pull request as ready for review February 21, 2023 16:48
@Haroenv Haroenv requested review from a team, FabienMotte and aymeric-giraudet and removed request for a team February 21, 2023 16:48
@Haroenv Haroenv changed the title feat(insights): option to disable feat(insights): option to disable automatic insights middleware Feb 21, 2023
@Haroenv Haroenv removed the request for review from aymeric-giraudet February 22, 2023 11:19
Base automatically changed from feat/automatic-insights-middleware to feat/automatic-insights February 23, 2023 10:56
Haroenv and others added 3 commits February 23, 2023 11:59
@Haroenv Haroenv force-pushed the feat/automatic-insights-option branch from a15c0bc to c76dd8e Compare February 23, 2023 10:59
@Haroenv Haroenv merged commit d2d6622 into feat/automatic-insights Feb 23, 2023
@Haroenv Haroenv deleted the feat/automatic-insights-option branch February 23, 2023 11:00
Haroenv added a commit that referenced this pull request Mar 2, 2023
FX-2250 

Co-authored-by: Sarah Dayan <5370675+sarahdayan@users.noreply.github.com>
Haroenv added a commit that referenced this pull request Mar 2, 2023
FX-2250 

Co-authored-by: Sarah Dayan <5370675+sarahdayan@users.noreply.github.com>
Haroenv added a commit that referenced this pull request Mar 8, 2023
FX-2250 

Co-authored-by: Sarah Dayan <5370675+sarahdayan@users.noreply.github.com>
Haroenv added a commit that referenced this pull request Apr 18, 2023
FX-2250 

Co-authored-by: Sarah Dayan <5370675+sarahdayan@users.noreply.github.com>
Haroenv added a commit that referenced this pull request Apr 24, 2023
FX-2250 

Co-authored-by: Sarah Dayan <5370675+sarahdayan@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants