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

Ember 5 support? #1906

Closed
petenickless opened this issue Nov 13, 2023 · 6 comments · Fixed by #1908
Closed

Ember 5 support? #1906

petenickless opened this issue Nov 13, 2023 · 6 comments · Fixed by #1908

Comments

@petenickless
Copy link

Hello,

Just wondering if there are any plans to upgrade this plugin for use with Ember 5?

Thanks

@RobbieTheWagner
Copy link
Owner

@petenickless is it not currently working with Ember 5?

@AmauryD
Copy link
Contributor

AmauryD commented Nov 13, 2023

We are using Ember 5 with embroider and we have no issue with ember flatpickr. Maybe you mean a V2 addon format ?

@petenickless
Copy link
Author

Hey @AmauryD @RobbieTheWagner - thanks for getting back to me.

We're getting ERESOLVE unable to resolve dependency tree when running ember install ember-flatpickr


🚧  Installing packages... This might take a couple of minutes.
Command failed with exit code 1: npm install --save-dev --loglevel error ember-flatpickr
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving: boma@0.0.0
npm ERR! Found: ember-source@5.4.0
npm ERR! node_modules/ember-source
npm ERR!   dev ember-source@"~5.4.0" from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer ember-source@"^3.28.0 || ^4.0.0" from ember-flatpickr@4.0.0
npm ERR! node_modules/ember-flatpickr
npm ERR!   dev ember-flatpickr@"*" 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.
npm ERR!
npm ERR! See /Users/pete/.npm/eresolve-report.txt for a full report.

npm ERR! A complete log of this run can be found in:
npm ERR!     /Users/pete/.npm/_logs/2023-11-13T19_53_39_422Z-debug-0.log```

We're running ember-cli v5.4.0.  Are you guys up to date also?

Any help would be gratefully received.  

Thanks

@AmauryD
Copy link
Contributor

AmauryD commented Nov 14, 2023

@petenickless

The problem is from the dependency resolution. Ember-flatpickr allows Ember 3.x and Ember 4.x for its peerDependencies. Npm seems to be strict about this and throws an error because you have Ember 5 installed. I'll submit a pull request to fix this issue.

Because i'm not using pnpm in 'strict' mode, i only have a warning on my side.
image

@petenickless
Copy link
Author

Thanks so much @AmauryD - very much appreciated.

@petenickless
Copy link
Author

Very much appreciate your efficiency on this @RobbieTheWagner and @AmauryD. Thanks again.

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.

3 participants