Skip to content

effectful initial state#92

Open
martyall wants to merge 2 commits intopaf31:masterfrom
martyall:effectful-initial-state
Open

effectful initial state#92
martyall wants to merge 2 commits intopaf31:masterfrom
martyall:effectful-initial-state

Conversation

@martyall
Copy link

This pr is a breaking change that allows for effectful actions while retrieving initial state for the component.

@martyall
Copy link
Author

I just noticed all the package-lock changes made it in, should I take them out?

@paf31
Copy link
Owner

paf31 commented Nov 14, 2017

That file should just be deleted actually. Would you mind?

@martyall
Copy link
Author

done

@paf31
Copy link
Owner

paf31 commented Nov 19, 2017

Thanks! I plan to merge this as a breaking change along with updates for the latest react package.

@athanclark
Copy link
Collaborator

athanclark commented Feb 20, 2019

If I'm not mistaken, in React v16.x, there is no ability to create a component without an immediate, pure initial state - https://pursuit.purescript.org/packages/purescript-react/6.1.0/docs/React#t:ReactSpecRequired - ping @blinky3713

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants