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

Me: Create flux stores for CurrentUser and CurrentUserSettings #462

Closed
alisterscott opened this issue Nov 23, 2015 · 1 comment
Closed

Me: Create flux stores for CurrentUser and CurrentUserSettings #462

alisterscott opened this issue Nov 23, 2015 · 1 comment
Labels
[Feature] User & Account Settings (/me) Settings and tools for managing your WordPress.com user account. State [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it

Comments

@alisterscott
Copy link
Contributor

Originally raised by @ebinnion

I think two Flux stores: one for CurrentUser and one for CurrentUserSettings would solve this because the CurrentUser store could be updated when the setting response resolves instead of having to do a separate fetch or some sort of direct mutation of the user object.
@alisterscott alisterscott added [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it [Feature] User & Account Settings (/me) Settings and tools for managing your WordPress.com user account. Flux labels Nov 23, 2015
@lancewillett
Copy link
Contributor

Closing as this is obsolete now (not using Flux for state).

@snyk-bot snyk-bot mentioned this issue Sep 25, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
[Feature] User & Account Settings (/me) Settings and tools for managing your WordPress.com user account. State [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lancewillett @alisterscott