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

Add support for keepAlive #1949

Open
1 of 2 tasks
weiwei-lin opened this issue Aug 23, 2022 · 1 comment
Open
1 of 2 tasks

Add support for keepAlive #1949

weiwei-lin opened this issue Aug 23, 2022 · 1 comment
Labels
enhancement Possible enhancement help/PR welcome Help/Pull request from contributors to fix the issue is welcome level: intermediate

Comments

@weiwei-lin
Copy link

Feature request
Support equivalent of @computed({keepAlive: true}).
Note that keepAlive from mobx-utils isn't the same. Since it causes the computed value to always be re-evaluated even there's no other observers.

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

The lack of computed support make the friction of migrating from MobX to MST a lot higher.
And given that MST use a tree-like structure for managing states, arguably the chance of memory leak is even lower.

Are you willing to (attempt) a PR?

  • Yes
  • No
@coolsoftwaretyler
Copy link
Collaborator

@weiwei-lin - kind of like I mentioned in #1950 - are you still having trouble with this/interested in opening a PR? I don't know if I understand all the implications of this feature request, but I'd be happy to talk with you about it if it's still on your mind all this time later.

@coolsoftwaretyler coolsoftwaretyler added enhancement Possible enhancement help/PR welcome Help/Pull request from contributors to fix the issue is welcome level: intermediate labels Jun 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Possible enhancement help/PR welcome Help/Pull request from contributors to fix the issue is welcome level: intermediate
Projects
None yet
Development

No branches or pull requests

2 participants