FormidableLabs / freactal

Clean and robust state management for React and React-like libs.
MIT License
1.65k stars 46 forks source link

fix(README): synchronous use of event in effect #92

Closed julien-f closed 6 years ago

julien-f commented 6 years ago

The event must be used directly in the effect, not in the async state updater.

I could have used directly param destructuring to get the value but I thought it would complexify things unnecessarily.


The maintainers of this repo require that all pull request submitters adhere to the following:

The maintainers of this repository require you to select the semantic version type that the changes in this pull request represent. Please select one of the following:

maintainerd[bot] commented 6 years ago

maintainerd logging is enabled for this repository.

All actions related to rules and their enforcement will be logged here as a permanent record.


Click to view log...

- `2018-01-04T14:42:43.093Z:abbc1fa`: The pull request was created - `2018-07-06T08:58:46.215Z:abbc1fa`: @divmain selected `documentation only` as the semantic version.