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

[CLEANUP] removed providing reversed arguments to observer #15893

Merged
merged 1 commit into from
Dec 1, 2017

Conversation

bekzod
Copy link
Contributor

@bekzod bekzod commented Nov 30, 2017

No description provided.

@bekzod bekzod mentioned this pull request Nov 30, 2017
34 tasks
@bekzod bekzod force-pushed the remove-reverse-args-observer branch from ac89968 to 1f71415 Compare November 30, 2017 09:29
@@ -842,22 +833,12 @@ export function _immediateObserver() {
@private
*/
export function _beforeObserver(...args) {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed providing reversed arguments to _beforeObserver too

@bekzod bekzod force-pushed the remove-reverse-args-observer branch from 1f71415 to e383e1d Compare November 30, 2017 18:02
@rwjblue
Copy link
Member

rwjblue commented Dec 1, 2017

Can you create a corresponding PR (or issue that links back here) at https://github.com/emberjs/ember-2-legacy? All of the API functionality we remove from this repo for 3.0 must be "re-enableable" by including that addon.

@bekzod
Copy link
Contributor Author

bekzod commented Dec 1, 2017

done

@rwjblue rwjblue merged commit 3844b42 into emberjs:master Dec 1, 2017
@rwjblue
Copy link
Member

rwjblue commented Dec 1, 2017

Thank you!

@bekzod bekzod deleted the remove-reverse-args-observer branch December 1, 2017 13:54
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

Successfully merging this pull request may close these issues.

2 participants