-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Deprecation notes #5821
Deprecation notes #5821
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good other than one duplicated bit.
Related to discussion here: #5824 |
f389716
to
d5ac0fc
Compare
I updated this pr to align with #5824 and also I created a page for the breaking changes |
…forkJoin and combineLatest
…-selector operators
25fa017
to
c265931
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm merging this 'cause it's a docs-only change, 'cause the deprecations effect an inordinate amount of confusion and need to be sorted out and because keeping a PR that's been open for 6 months in sync with the changes that accrue in the branch it targets is a PITA.
FWIW, over the next few days, I'll be working on fixing some issues with deprecations and that will be much easier now that this PR has been merged. |
This are the important deprecation notes from my point of view. We introduced some breaking changes recently, but I personally would keep these information in the changelog, those feel very edgy to me.
closes #4776 ?