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

Fix deprecations regarding Computed Property Overrides #2076

Merged
merged 5 commits into from
Jan 31, 2020
Merged

Conversation

marcoow
Copy link
Member

@marcoow marcoow commented Dec 30, 2019

This fixes the deprecations regarding overridden computed properties. Most of these CPs were only used in testing and should be properly mocked via the container anyway (e.g. _fastBoot).

TODO

  • all deprecations in app code should be fixed but there are some left in the testing code (mainly the code around testing adaptive and cookie session stores which is relatively messy anyway 😞) - that should be fixed before releasing this as well.

closes #1908
closes #2044
closes #1938

@marcoow marcoow marked this pull request as ready for review January 31, 2020 16:34
Co-Authored-By: Samanta de Barros <samanta.debarros@simplabs.com>
@marcoow marcoow merged commit 5f2c636 into master Jan 31, 2020
@delete-merged-branch delete-merged-branch bot deleted the fix-cp-override branch January 31, 2020 17:09
@pjcarly
Copy link

pjcarly commented Feb 4, 2020

Great, thanks for the merge.

Could we get a new NPM release?

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

Successfully merging this pull request may close these issues.

Deprecation warnings on Ember 3.9 (computed property was just overriden)
3 participants