Skip to content
This repository has been archived by the owner on Feb 4, 2023. It is now read-only.

Migrate away from MobX #76

Closed
shilangyu opened this issue Oct 25, 2020 · 0 comments · Fixed by #79
Closed

Migrate away from MobX #76

shilangyu opened this issue Oct 25, 2020 · 0 comments · Fixed by #79
Assignees
@shilangyu
Labels
enhancement New feature or request Priority: medium Will be implemented

Comments

@shilangyu
Copy link
Collaborator

Is your feature request related to a problem? Please describe.

While MobX provides nice abstractions and reactivity, it introduces a lot of maintenance burden. Even though the rules for MobX reactivity are clearly stated, in reality it is very unpredictable. Currently some properties do not react to changes. Fixing it is possible, but this is a recurring theme and a lot of time is wasted on fixing it.

Describe the solution you'd like

Migrating away from MobX for global state managment

Describe alternatives you've considered

  • Provider + ChangeNotifier
  • Provider + Hooks
  • BLoC
  • states_rebuilder

List some pros and cons for each choice
@shilangyu shilangyu added enhancement New feature or request Priority: medium Will be implemented labels Oct 25, 2020
@shilangyu shilangyu mentioned this issue Oct 25, 2020
Merged
@shilangyu shilangyu self-assigned this Oct 25, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@shilangyu shilangyu

Labels
enhancement New feature or request Priority: medium Will be implemented
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Replaced MobX with ChangeNotifier LemmurOrg/lemmur
1 participant
@shilangyu