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

Optimistic updates #12

Open
tappleby opened this issue Aug 9, 2015 · 3 comments
Open

Optimistic updates #12

tappleby opened this issue Aug 9, 2015 · 3 comments

Comments

@tappleby
Copy link
Contributor

tappleby commented Aug 9, 2015

This is related to the sequence discussion in redux-utilities/flux-standard-action#7.

If payload is a Promise, where should data needed to perform the optimistic update be put? meta?

Wherever this data ends up being stored I think it should become the payload for first action in sequence.

@acdlite
Copy link
Contributor

acdlite commented Aug 14, 2015

We're using the sequence branch of redux-promise and redux-actions at OpenGov, and we put data needed for optimistic updates inside of meta. Not sure about making it the payload of the first action — it seems redundant, if it's already part of meta.

@tappleby
Copy link
Contributor Author

Makes sense, my initial though was it felt weird having to look at meta while in the reducer instead of payload. After thinking about it some more, it would probably be confusing for a value you put in meta to move to payload.

@kwelch
Copy link

kwelch commented Jul 21, 2016

Is there an npm version that be installed to leverage sequence updates prior to it being part of spec?

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

No branches or pull requests

3 participants