-
-
Notifications
You must be signed in to change notification settings - Fork 103
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
Remove StateObject:get()
from the public API
#168
Labels
Comments
Related to #111 |
This was referenced Jul 7, 2022
Is there going to be any difference between this and |
See #217 for an explanation of the differences. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Alternate idea to #159 - since we're introducing
unwrap()
in v0.3, users will already have a great consistent way to get the value of any state object. There's no case where usingunwrap()
would be disadvantageous over using:get()
, and generally using:get()
locks your code into only using state objects, so the case for keeping this method public is rather vague and weak.We'll still need a method like this to exist, of course -
unwrap()
needs someplace to source its values, after all - but perhaps that is better left as a lower level implementation detail that only the authors of state objects need to worry about, not general users of Fusion.The text was updated successfully, but these errors were encountered: