@jameshahn2 wrote:
Tweeted @lukemelia a month ago asking if he planned to update
ember-modal-dialogto remove this deprecation warning.#targetAttachmentClass computed property was just overriden. This removes the computed property and replaces it with a plain value, and has been deprecated.Was looking into it last night and saw someone submitted an issue about the deprecation around the same time. Told Luke if someone could guide me through it, I’d be happy to submit the PR. If you follow the link, you will see his reply:
Hi @jameshahn2, the reason for this error is that targetAttachmentClass is a computed property. If a value is passed in for the property, Ember’s historical behavior is that the passed value would replace the computed function. This is now deprecated, hence the warning. So we need a new way to provide a default value for targetAttachmentClass. I think that a way to do it would be to make the init method of the component(s) set targetAttachmentClass if and only if this.targetAttachmentClass is undefined.
After searching the repo for “target”, I narrowed the problematic files down to what you see below.
This is not necessarily every file that needs to be fixed and there could be far fewer problematic files than I linked to here. However, I’m hoping they will help y’all see what needs to happen. If a couple of you could take a look and tell me how to fix the issue, I will
git clonethe directory, make the changes, and submit the PR.Let’s make
ember-modal-dialoggreat again!
Posts: 1
Participants: 1