Breaking changes in Mobx 6

Unfortunately there has been a breaking change with mobx 6 - makeObservable(this) will have to be added to the constructor of any classes that are using the @observable, @action or @computed decorators.

More information and example here: https://mobx.js.org/enabling-decorators.html

Alternatively the decorators can be removed entirely and makeObservable can be used with a second parameter, which is described in the upgrade guide: https://mobx.js.org/migrating-from-4-or-5.html#upgrading-classes-to-use-makeobservable

I’m not sure which method you would prefer?

Cheers, Felix

[vkrol]

@foxhound87 @ajorkowski What do you think about upgrading Babel to 7, MobX to 6 and releasing a new major version of mobx-react-form without support for older versions of MobX? I did it in my fork and I can create the PRs (all tests are green except the performance test).

1. Upgrade Babel to 7: https://github.com/foxhound87/mobx-react-form/compare/master...vkrol:upgrade-babel-to-7
2. Upgrade MobX to 6: https://github.com/vkrol/mobx-react-form/compare/upgrade-babel-to-7...vkrol:upgrade-mobx-to-6

[foxhound87]

The mobx6 support has been released in version 3