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 >= 4.x compatibility (require ember-source 4.8+) #22

Conversation

void-mAlex
Copy link
Contributor

motivation for this is that @ember/legacy-built-in-components is a peer dep for ember-engines for "ember-source": "^3.24.1 || 4",
in ember-source: 4 sendAction was rightly cleaned up and the named export is no longer resolvable which can cause warnings at best and errors at worst (trying to run an ember-engine while using a LinkTo component on ember source 4.x)

with this PR I propose we close that gap by releasing a new major for @ember/legacy-built-in-components to be used with ember-source >= 4.x

@void-mAlex void-mAlex changed the title remove dprecated features that were dropped in 4.x remove deprecated features that were dropped in 4.x Aug 22, 2023
@SergeAstapov
Copy link
Contributor

tbh not sure it worth the churn and effort. maybe I'm missing if there is an actual issue this PR fixes?

@void-mAlex
Copy link
Contributor Author

the short answer is I believe it to be worth the effort.
the actual issue is with running engines with embroider under vite. this is an error in that case and your app doesn't load at all because the import it's doing is not valid on ember-source > 3.x
the alternative would be to pr a change to engines to work without LinkTo component for example or any of the components provided by this addon

@SergeAstapov
Copy link
Contributor

the actual issue is with running engines with embroider under vite

sounds reasonable!

the alternative would be to pr a change to engines to work without LinkTo component for example or any of the components provided by this addon

it means copy-pasting pasts of this addon.
we tried to brainstorm ideas how to move forward without using something old LinkTo component and came up with emberjs/rfcs#819 which was put on hold as there is larger router initiative as part of Polaris.

@SergeAstapov
Copy link
Contributor

I wonder if @mixonic is ok to land this and get new major out.

@void-mAlex I would say, we need to provide peerDeps on ember-source: >= 4 then + update readme as well as ember-try scenarios?

@void-mAlex
Copy link
Contributor Author

ok will try and get those done

regarding the failing ember-try scenario for ember-beta
I think either adding @glint/template to dev dependencies of the addon or configuring the scenario to install it for that run would fix it
any preference on the approach?

@SergeAstapov
Copy link
Contributor

SergeAstapov commented Aug 23, 2023

@void-mAlex either should be fine IMO. Thank you!

drop node 12 and 14
@void-mAlex void-mAlex changed the title remove deprecated features that were dropped in 4.x ember >= 4.x compatibility Oct 7, 2023
@void-mAlex
Copy link
Contributor Author

@mixonic @SergeAstapov I managed to get ci green as well
bumped peer declaration to be ember-source >=4
updated readme to reflect that
ember-beta is running against 5.x series so the loose peer declaration should be correct/ok to leave open for future ember version

this would unblock engines continuing to work through the 4.x and 5.x ember source version especially through embrodier

let me know if I missed something or anything else I need to do to land this.
thank you in advance

@SergeAstapov SergeAstapov merged commit 095fa3e into emberjs:main Oct 11, 2023
@void-mAlex void-mAlex deleted the remove-sendAction-and-string-action-passing branch October 11, 2023 11:25
@SergeAstapov SergeAstapov changed the title ember >= 4.x compatibility Ember >= 4.x compatibility (require ember-source 4.8+) Mar 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants