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

[DEPRECATE] - RFC 421 - deprecate app ctrl router props #17573

Merged

Conversation

mistahenry
Copy link
Contributor

@mistahenry mistahenry commented Feb 6, 2019

Implements RFC #421.
Part of #17569.

Please let me know if notifyPropertyChange was an okay way to solve this.

@locks
Copy link
Contributor

locks commented Feb 7, 2019

Can you add the following URL to the deprecation, https://emberjs.com/deprecations/v3.x#toc_application-controller-router-properties?
It should become valid when ember-learn/deprecation-app#276 gets merged.

@mistahenry mistahenry force-pushed the deprecate/app-ctrl-route-props branch from 374865a to b43cf34 Compare February 7, 2019 14:41
@mistahenry
Copy link
Contributor Author

@locks Updated the deprecation with the url/ matching deprecation id

Copy link
Contributor

@chadhietala chadhietala left a comment

Choose a reason for hiding this comment

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

This looks great! Thanks for doing this, just a couple small things we need to address before merging.

… currentRoute and currentRouteName properties
@mistahenry mistahenry force-pushed the deprecate/app-ctrl-route-props branch from b2f888f to ce7841f Compare February 14, 2019 16:30
@chadhietala chadhietala merged commit fe93892 into emberjs:master Feb 15, 2019
@chadhietala
Copy link
Contributor

Thanks so much @mistahenry!

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.

3 participants