microsoft/satcheljs

Switch to new version of mobx

Closed this issue · 5 comments

KatSick commented

Now with release 3.1.0 we can install latest mobx, however some warnings occurred like

[mobx] Deprecated: `mobx.map` is deprecated, use `new ObservableMap` or 
`mobx.observable.map` instead

I will try my best to create MR for this

mohamedmansour commented

Are you still working on this? There is a warning that satcheljs has incorrect peer dependency
"mobx@^4.4.0"

mohamedmansour commented

@smikula @kenotron any plans to update to mobx 5? Stay upto date with latest libraries

MLoughry commented

MobX v5 won’t run on Internet Explorer; so until Outlook Web can drop support for IE, I don’t think we’re taking on MobX v5.

smikula commented

Exactly this. My understanding is that MobX v4 is API compatible with v5, so it should be possible to swap in the newer version if you know your browsers support it. (Or even build different bundles for different browsers with the appropriate version, if you're ambitious.) But we haven't specifically tested against v5, so I can't promise everything will work. (Feel free to try it and open issues if you find any!)

maheshp9 commented

Is there a plan to to release satcheljs 4.x release (out of beta/pre-release) ?