Clean up flux interfaces

google / flutter_flux

Implementation of the Flux framework for Flutter

https://flutter.io

Apache License 2.0

372 stars 36 forks source link

Closed abarth closed 7 years ago

abarth commented 8 years ago

Rather than using complex mixin inheritance, we now use composition for StoreListener.
Rename StoreWatcher.initState to StoreWatcher.initStores to suggest that this function is for initializing the stores rather than initializing the (hidden) state object.
Update the dartdocs to reflect how the code works currently.

abarth commented 8 years ago

@jimbeveridge

abarth commented 7 years ago

Thanks. I've switching this back to being a mixin. Please take another look.

jimbeveridge commented 7 years ago

LGTM. Thanks for your work on this!

The example app in the repo is broken due to missing pubspec.yaml. I will fix that tomorrow.

abarth commented 7 years ago

Thanks!