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

autocomplete-plugin-algolia-insights can not be installed alongside search-insights@2.x #929

Closed
salomvary opened this issue Apr 1, 2022 · 1 comment · Fixed by #930
Closed

Comments

@salomvary
Copy link
Contributor

Description

Even though search-insights@2 does not seem to contain any breaking changes that are truly incompatible with autocomplete-plugin-algolia-insights, npm refuses to install the latest versions of both the same time.

Reproduction

$ npm --version
8.3.1
$ npm i @algolia/autocomplete-plugin-algolia-insights search-insights@2
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR! 
npm ERR! While resolving: undefined@undefined
npm ERR! Found: search-insights@2.2.1
npm ERR! node_modules/search-insights
npm ERR!   search-insights@"2" from the root project
npm ERR! 
npm ERR! Could not resolve dependency:
npm ERR! peer search-insights@"^1.0.0" from @algolia/autocomplete-plugin-algolia-insights@1.5.6
npm ERR! node_modules/@algolia/autocomplete-plugin-algolia-insights
npm ERR!   @algolia/autocomplete-plugin-algolia-insights@"*" from the root project
npm ERR! 
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

Solution

Merge and release #717

@Haroenv
Copy link
Contributor

Haroenv commented Apr 1, 2022

For the time being, you can install with --legacy-peer-deps, but indeed this should be fixed

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 a pull request may close this issue.

2 participants