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
replace ember-render-modifiers dependency #5374
Labels
Comments
ballpark
|
stopfstedt
changed the title
replace ember-render-modifiers
replace ember-render-modifiers dependency
Apr 2, 2024
I was looking at a template and VSCode was complaining about this, so I searched to see if anyone else noticed, and lo and behold. I did not realize that there were so many instances of it, though :-O |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
our template linting library now actively warns against the usage of
{{did-insert}}
,{{did-update}}
,{{will-destroy}}
library, see the linting rules ref entry for the motivation behind this https://github.com/ember-template-lint/ember-template-lint/blob/master/docs/rule/no-at-ember-render-modifiers.mdthose modifiers are provided by https://github.com/emberjs/ember-render-modifiers.
we're currently using (abusing?) these modifiers by attaching tasks to them mainly for loading/reloading async data during component initialization.
this is a legacy pattern. we've already moved off of it in several places, using a tracked resources provided by https://github.com/tracked-tools/ember-async-data instead.
however, we still have many components left that are currently adhering to the render-modifier based approach.
replace all remaining instances.
The text was updated successfully, but these errors were encountered: