Skip to content

Unit tests for issue #149 #153

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.

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

Merged
merged 4 commits into from
Jul 21, 2016

Conversation

e-schultz
Copy link
Member

No description provided.

@e-schultz
Copy link
Member Author

@SethDavenport @danielfigueiredo related to an issue a user raised - any idea? made some tests to reproduce the issue. Initially I thought it might have been something related to the angular lifecycle hooks, but was able to reproduce it outside of angular.

this._store.subscribe(() => this._store$.next(this._store.getState()));
let x = store as any;

this._store$ = Observable.from(x); //new BehaviorSubject(store.getState());
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@SethDavenport was looking at the interop unit tests from redux

this seems to fix the issue, although still bugging me to what the problem was with the BehaviorSubject

Copy link
Member

@SethDavenport SethDavenport Jul 12, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Weird - I wonder if there's a subtle difference in 'zoning' between the two ways of creating observables.

@e-schultz e-schultz force-pushed the bug-action-dispatch-order branch 2 times, most recently from 69a2d60 to 9b49d6e Compare July 14, 2016 16:53
Evan Schultz added 2 commits July 19, 2016 16:55
* fix angular-redux#149 - chained actions should not get stale state,
  changed to create an observable from redux since redux 3.4x supports
  an observable shim
* fix angular-redux#138 - ability to use select decorators in service
  Changing to use redux's observable, had to change how we created the
  initial observable. Use switchMap to switch streams once the store
  observable becomes available
@e-schultz e-schultz force-pushed the bug-action-dispatch-order branch from 2c3f554 to 381228a Compare July 21, 2016 15:56
@e-schultz e-schultz merged commit 6237dbc into angular-redux:master Jul 21, 2016
e-schultz pushed a commit that referenced this pull request Jul 21, 2016
* 3.3.0

* Features
 * DevToolsExtension - convience wrapper for dev tools (#115)
 * Select - seamless support for ImmutableJS (#160)

* Fixes
 * Able to use `@select` in services
 * Behavior of `select` with chained dispatches, (fixes #149, #153)
@e-schultz e-schultz mentioned this pull request Jul 21, 2016
e-schultz pushed a commit that referenced this pull request Jul 21, 2016
* 3.3.0

* Features
 * DevToolsExtension - convience wrapper for dev tools (#115)
 * Select - seamless support for ImmutableJS (#160)

* Fixes
 * Able to use `@select` in services
 * Behavior of `select` with chained dispatches, (fixes #149, #153)
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.

2 participants