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

sendAction deprecation #614

Closed
josemarluedke opened this issue Aug 27, 2018 · 7 comments
Closed

sendAction deprecation #614

josemarluedke opened this issue Aug 27, 2018 · 7 comments

Comments

@josemarluedke
Copy link
Contributor

Running ember-table with Ember v3.4 beta gave me the following deprecation:

DEPRECATION: You called <my-app@component:ember-tr::ember7583>.sendAction("onClick") but Component#sendAction is deprecated. Please use closure actions instead. [deprecation id: ember-component.send-action] See https://emberjs.com/deprecations/v3.x#toc_ember-component-send-action for more details.
@pzuraq
Copy link
Contributor

pzuraq commented Aug 27, 2018

Unfortunately due to our support matrix (we support Ember 1.12+) we won’t be able to change this API until Ember Table v3. We should look to see if there is a way to suppress this deprecation until then.

@Alonski
Copy link

Alonski commented Sep 14, 2018

Isn't there a way to check what version the app is running and call functions accordingly?
Do closure actions and legacy actions at the same time somehow?

@pzuraq
Copy link
Contributor

pzuraq commented Sep 14, 2018

There is ember-compatability-helpers which would allow you to do that, happy to merge PRs but I won’t have time to continue working on this project for the forseeable future. Main thing that will be tricky is tests

@Alonski
Copy link

Alonski commented Sep 14, 2018

@pzuraq That looks interesting. If I find the time I will check it out. Thanks

@sanmaverick
Copy link

Are there any plans for fixing this in ember table v2?

@HeroicEric
Copy link
Contributor

I believe this was fixed by #860, so this can probably be closed

@kpfefferle
Copy link
Member

Resolved by #860

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
6 participants