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

Separate IDispatcher / IStore #209

Closed
mrpmorris opened this issue Sep 13, 2021 · 0 comments
Closed

Separate IDispatcher / IStore #209

mrpmorris opened this issue Sep 13, 2021 · 0 comments
Milestone
5.0

Comments

@mrpmorris
Copy link
Owner

People frequently try to inject IDispatcher into dependencies indirectly referenced by the store

e.g. Store => Effect => ServiceThatNeedsIDispatcher

This currently doesn't work because IDispatcher is an alias for IStore. The proposal is to separate IDispatcher and IStore.
@mrpmorris mrpmorris added this to the 5.0 milestone Nov 3, 2021
mrpmorris added a commit that referenced this issue Nov 25, 2021
@mrpmorris
Separate IDispatcher / IStore #209
bd35e4f
mrpmorris added a commit that referenced this issue Nov 25, 2021
@mrpmorris
Separate IDispatcher / IStore #209 (#237)
a252915
mrpmorris added a commit that referenced this issue Jan 13, 2022
@mrpmorris
Release/5.0 (#239)
65a96b5 
* Remove need for index.js #235 (#236)

* Remove need for index.js (#235)

* Update to V5

* Separate IDispatcher / IStore #209 (#237)

* IStateSelector<TState, TValue> #221 (#238)

* Language version 9 features

* Added 5.0 breaking changes to releases

* Fix JS accidentally introduced error in ReduxDevTools middleware

* Make generic action types human readable in ReduxDevTools #205 (#249)

* Make generic action types in ReduxDevTools human readable #205

* Move actions to common folder in tutorials #250 (#251)

* Update versions to 5.0.0
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
5.0
Development

No branches or pull requests
1 participant
@mrpmorris