-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Flux to Redux: Plans, Domains & Purchases #6144
Comments
I added a checkbox to this and #5046 to add a Edit: Actually, the current Flux store is a reducer, so we can probably just move/update it. |
@scruffian is "Remove plans flux stores" done or still pending? Removing |
The last checkbox for plans is still pending... |
Just FYI, there's already a Edit: for reference, here are the two related PRs where I recently removed calls to this old store: #46418 and #47060 |
This issue is stale because it has been 180 days with no activity. You can keep the issue open by adding a comment. If you do, please provide additional context and explain why you’d like it to remain open. You can also close the issue yourself — if you do, please add a brief explanation and apply one of relevant issue close labels. |
As outlined in #5046 we need to move the
/plans
,/purhcases
and/domains
to Redux. The work can be broken down like this:PlansQuerySitePlans
component. Framework: add query plans component #4933QueryPlans
component. Framework: add QueryPlans data component #5444Domains
QueryDomains
component.StoreConnection
in domain management.Purchases
client/lib/purchases/reducer
toclient/state/
Purchases: Addstate.purchases.items
reducer #6317QueryPurchases
component. Purchases: Addstate.purchases.items
reducer #6317StoreConnection
in purchases. Purchases: Addstate.purchases.items
reducer #6317state.purchases
to render/purchases
#6647This issue is to investigate how much work this will be and how we will approach it.
The text was updated successfully, but these errors were encountered: