Model architecture: Configure mobx to disallow observable access outside reactive context

TerriaJS / terriajs

A library for building rich, web-based geospatial data platforms.

https://terria.io

Apache License 2.0

1.15k stars 356 forks source link

Model architecture: Configure mobx to disallow observable access outside reactive context #3488

Open kring opened 5 years ago

kring commented 5 years ago

This will break things, but once we fix them, it will help detect a whole class of common errors (e.g. accessing observables inside a promise callback).

na9da commented 4 years ago

Partially fixed by #3616. Need to fix console warnings because of enabling computedRequiresReaction