-
Notifications
You must be signed in to change notification settings - Fork 819
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
docs: creating nested models #346
base: main
Are you sure you want to change the base?
Conversation
I would suggest we should maybe make an |
c69903c
to
515951e
Compare
hey @maaslalani great suggestion. I actually found some typos while moving the code over to an example 😂 what do you think about these changes? I feel like these docs will become more substantial as we get more questions and as we get feedback. Just pushed the latest changes |
Looks awesome @bashbunni !!! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few quick notes here:
- Please do make sure the example code runs (I'm sure it's still a work-in-progress)
- It would be great to show how one can use a state member in
Update
, in addition toView
. For example, to change key handling when a section of the app changes focus, since that will be a super common need.
fa23d0b
to
41da988
Compare
tbd in dev meeting. I'd like some feedback on writing effective comments in the code and whether or not this example should use |
ec0f4a4
to
d949506
Compare
This example no longer manages state from a main model. That is shown in the |
d949506
to
56acacc
Compare
Changes: