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

Deprecation warnings with Ember 3.26+ #319

Closed
pgengler opened this issue Apr 11, 2021 · 5 comments
Closed

Deprecation warnings with Ember 3.26+ #319

pgengler opened this issue Apr 11, 2021 · 5 comments

Comments

@pgengler
Copy link

pgengler commented Apr 11, 2021

Beginning with Ember 3.26, implicit injections are deprecated, leading to deprecation warnings like this:

A value was injected implicitly on the 'destinationElementId' property of an instance of <app-name@service:modal-dialog::ember616>, overwriting the original value which was null. Implicit injection is now deprecated, please add an explicit injection for this value. If the injected value is a service, consider using the @service decorator.

Additionally, this.-fallback in templates is deprecated:

The modalDialogComponentName property path was used in a template for the ember-modal-dialog/templates/components/modal-dialog.hbs component without using this. This fallback behavior has been deprecated, all properties must be looked up on this when used in the template: {{this.modalDialogComponentName}}

(Similarly for attachment, attachmentClass, clickOutsideToClose, constraints, containerClass, containerClassNames, destinationElementId, hasOverlay, offset, overlayClass, overlayClassNames, overlayPosition, stack, target, targetAttachment, targetAttachmentClass, targetModifier, targetOffset, tetherClassPrefix, tetherTarget, translucentOverlay, value, wrapperClass, wrapperClassNames, etc.)

This is happening with the latest release of ember-modal-dialog (3.0.2).

@alexlafroscia
Copy link

alexlafroscia commented Aug 25, 2021

It seems like this should be fixed on master; @lukemelia is it possible to cut a release that includes it?

Edit: It looks like the alpha release includes it

https://www.npmjs.com/package/ember-modal-dialog/v/4.0.0-alpha.0

@bartocc
Copy link

bartocc commented Jan 13, 2022

Hi 👋 any news on a possible 3.0.4 release?

@lukemelia
Copy link
Contributor

@bartocc can you try 4.0.0-alpha.1 and see if fixes this issue for you? If so, I can promote it to a 4.0.0 release.

@bartocc
Copy link

bartocc commented Jan 17, 2022

4.0.0-alpha.1 works for us @lukemelia 👍

@lukemelia
Copy link
Contributor

Closing as resolved. 4.0.0-beta.0 is out and we'll hopefully have 4.0.0 stable this week.

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

No branches or pull requests

4 participants