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.

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

Reader: Pull in stores that accept actions when pre-loading. #569

Merged
merged 1 commit into from
Nov 23, 2015

Conversation

blowery
Copy link
Contributor

@blowery blowery commented Nov 23, 2015

The Reader preloads all of a user's subscriptions when you hit any Reader URL. This works pretty well, but we have to ensure that all of the stores that want to listen for and act upon actions fired by the preload are spun up and listening.

This patch teaches the controller how to load all of those stores before kicking off the preload.

Fixes #559

The Reader preloads all of a user's subscriptions when you hit any Reader URL. This works pretty well, but we have to ensure that all of the stores that want to listen for and act upon actions fired by the preload are spun up and listening.

This patch teaches the controller how to load all of those stores before kicking off the preload.
@blowery blowery added [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. [Feature] Reader The reader site on Calypso. labels Nov 23, 2015
@blowery blowery self-assigned this Nov 23, 2015
@blowery
Copy link
Contributor Author

blowery commented Nov 23, 2015

cc @bluefuton

@bluefuton
Copy link
Contributor

👍 🚢

@bluefuton bluefuton added [Status] Ready to Merge and removed [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. labels Nov 23, 2015
bluefuton added a commit that referenced this pull request Nov 23, 2015
…dness

Reader: Pull in stores that accept actions when pre-loading.
@bluefuton bluefuton merged commit 2a9c77a into master Nov 23, 2015
@bluefuton bluefuton deleted the fix/following-notification-oddness branch November 23, 2015 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Reader The reader site on Calypso.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants