ember-eui icon indicating copy to clipboard operation
ember-eui copied to clipboard

Outdated version of Ember Power Select / Basic Dropdown

Open mkszepp opened this issue 1 year ago • 13 comments

Ember Power Select / Ember Basic Dropdown was moved ~ a year ago to an v2 addon (v8.x)

I have seen that the package is still using the legacy version v7.2.0

https://github.com/prysmex/ember-eui/blob/dcacaccbd379cb55432600c91f7ffbeecc549818/packages/core/package.json#L147

The move to v8 would be good, because there were improvement some parts of typing, a11y, dependencies were dropped and also updated.

In addtional you can drop one part of setup for this package (theme config is not needed) and move to recommended way to use it inside application.hbs.

https://github.com/prysmex/ember-eui/blob/dcacaccbd379cb55432600c91f7ffbeecc549818/docs/getting-started.md?plain=1#L85-L97

Here you can find the migration docs for this https://ember-power-select.com/docs/migrate-7-0-to-8-0

Changelogs: https://github.com/cibernox/ember-power-select/blob/master/ember-power-select/CHANGELOG.md

mkszepp avatar Jan 17 '25 06:01 mkszepp

Thanks @mkszepp ! will get on this soon. thanks for reporting.

We just need to be careful because consumers will get this dep bumped too, one time i got bitten because a component got deprecated in ember-power-select and ocnsumers of this addon got bitten by upgrading... even if this addon is not using any newer apis, the final users might

betocantu93 avatar Jan 23 '25 02:01 betocantu93

https://github.com/cibernox/ember-power-select/issues/1590

betocantu93 avatar Jan 23 '25 02:01 betocantu93

I think the update of this dependency is breaking, because the addon is shipping tight now as dependency.

I will recommend to move the dependency to peer so you can make updates in future without making always breaking, when there is a new power select version

mkszepp avatar Jan 26 '25 09:01 mkszepp

Yeah, making it a peer is a good idea!

betocantu93 avatar Jan 27 '25 18:01 betocantu93

Ok so I manage to make it work with ember-source 6.2.0, please give it a try on version, all packages on version 12.0.0.

Take a look to /site app if anything...

betocantu93 avatar Feb 07 '25 22:02 betocantu93

Seems that bumping power-select, again.. had breaking changes. Renaming helpers, moving apis around... these all should be majors on ember-power-select side.

betocantu93 avatar Feb 07 '25 22:02 betocantu93

@betocantu93 which helper was renamed in v8?

If you are talking about power-select-is-selected i don't give you right... it was not braking, because the helper was never offical public. It was never documented that it is public, marked inside class as prviate and in additional we did add this to release note after some people have reported that they were using. see It was more a copy / paste bug when people were overriding components

mkszepp avatar Feb 08 '25 07:02 mkszepp

fixed... ! 12.0.1 now has the latest and works.

betocantu93 avatar Feb 08 '25 20:02 betocantu93

Tahnks @mkszepp for reporting and follow-up

betocantu93 avatar Feb 08 '25 20:02 betocantu93

thank you for the updated release!

I tried a new project and I found this issue, I don't reopen the bug because I don't know if it's important or not, but I'll leave it here

WARN  Issues with peer dependencies found . └─┬ @ember-eui/core 12.0.1 └─┬ @ember/render-modifiers 2.1.0 └── ✕ unmet peer ember-source@"^3.8 || ^4.0.0 || ^5.0.0": found 6.2.0

Polve avatar Feb 08 '25 20:02 Polve

Intersting... supposedly render-modifiers has embersource >= 4.0.0

https://github.com/emberjs/ember-render-modifiers/blob/54f3c68c602ddbd5a56f98c7310ce884e30b92c4/package.json#L86

betocantu93 avatar Feb 08 '25 22:02 betocantu93

OKay seems the @ember/render-modifiers has pending changes to be released to npm. like a v3 or v2.2.0

in the long run: we should refactor this lib and instead use custom modifiers... this is extremely fragile and risky. Specially without proper testing suite.

betocantu93 avatar Feb 08 '25 22:02 betocantu93

A fix to this, if you are using pnpm... is using

pnpm patch @ember/render-modifiers

and patch the peerDependencies, to be >=4.0.0

betocantu93 avatar Feb 08 '25 22:02 betocantu93