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

Remove deprecated features for the next major version #3655

Closed
kubk opened this issue Mar 10, 2023 · 3 comments
Closed

Remove deprecated features for the next major version #3655

kubk opened this issue Mar 10, 2023 · 3 comments

Comments

@kubk
Copy link
Collaborator

kubk commented Mar 10, 2023

Since next Mobx update might be major due to native decorators support, it makes sense to revisit the API and remove deprecated features. Removing deprecated features gives less code to support for maintainers. For example here while mocking console.log I had to support both observer and useObserver usage:

function runTestSuite(mode: "observer" | "useObserver") {

Here is the list of deprecated API I've found:

Feel free to add missing API 👍 Once we agreed, I'll start working on PR.

@kubk kubk added 🐛 bug and removed 🐛 bug labels Mar 10, 2023
@urugator
Copy link
Collaborator

@mweststrate
Copy link
Member

The new decorators seem to be able to fit in a minor. Still, once they're out and stable for a while, it might be nice to push for a new major to remove the legacy decorators (and the makeObservable based api's?) to have a single uniform mechanism and make everything a bit leaner

@kubk
Copy link
Collaborator Author

kubk commented Nov 15, 2023

Closed in favor of #3796

@kubk kubk closed this as completed Nov 15, 2023
@mweststrate mweststrate mentioned this issue Nov 27, 2023
17 tasks
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

No branches or pull requests

3 participants