search

ds300 / derivablejs

Functional Reactive State for JavaScript and TypeScript
Apache License 2.0
515 stars 23 forks source link

Combine controller reactors #107

Closed TrySound closed 6 years ago

TrySound commented 6 years ago

For lifecycles you're create two reactors which are visible in captureDereferences. In this PR I just merged from lifecycle into reactor governor. Is it ok, @ds300 ?

TrySound commented 6 years ago

Also we can call lifecycle functions in a single derivation to reduce tree. What do you think?

ds300 commented 6 years ago

Looks good to me