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.

Sign up for GitHub

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

Jump to bottom

Component API: Mount, unmount, initialState #3

Closed
estrattonbailey opened this issue Jan 6, 2018 · 1 comment
Closed

Component API: Mount, unmount, initialState #3

estrattonbailey opened this issue Jan 6, 2018 · 1 comment

Comments

@estrattonbailey
Copy link
Owner

The component API could use mount and unmount, via on-load.

Is initialState() ok? Or should it just be an initial constructor? That's basically what it is, so maybe I'll call it init(). Could be considered a constructor for all intents and purposes, could even set state directly instead of returning state. Could also pry include a proxy of state so that you can't mutate it directly to begin with.
estrattonbailey added a commit that referenced this issue Jan 7, 2018
@estrattonbailey
initialState > init, contrib #3
26d785e
@estrattonbailey
Copy link
Owner Author

estrattonbailey commented Jan 7, 2018 •
edited
Loading

init() is complete, closing this in favor of #4

estrattonbailey added a commit that referenced this issue Jan 8, 2018
@estrattonbailey
initialState > init, contrib #3
36c4e25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@estrattonbailey