-
Notifications
You must be signed in to change notification settings - Fork 188
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
Is this the same as acdlite/redux-promise? #27
Comments
From what I can tell so far the major difference is, that Good point though, it probably makes sense to document it. I actually switched from acdlite/redux-promise due to the missing pending. |
I understand. Maybe the README should mention this in a sentence. For some time I thought I needed redux-promise to use redux-promise-middleware |
I have never used acdlite/redux-promise, but, yes, I would say the major difference is that this middleware dispatches a PENDING action. This PENDING action allows for optimistic updates, which is something acdlite/redux-promise has not yet implemented. I would also argue the API for this middleware is more transparent and easier to integrate, e.g., you do not need to use redux-actions. I'll update the README with the differences. Thanks for bringing this up. |
A year later, another difference became apparent, and for the benefit of community, someone should mention it. This project is actively maintained, while acdlite/redux-promise is not, judging by the number and age of unaddressed issues and PRs. |
What is the difference between the two projects? it should be explained somewhere in the documentation, it's confusing... are both the same?
The text was updated successfully, but these errors were encountered: