search

MichaelMarner / dart-redux-remote-devtools

Remote Devtools for Dart & Flutter
https://pub.dartlang.org/packages/redux_remote_devtools
MIT License
52 stars 10 forks source link

Add a note to the README that RemoteDevToolsMiddleware should be last #25

Closed nickmeinhold closed 4 years ago

nickmeinhold commented 4 years ago

I was seeing problems where Actions were listed out of order and State changes being reported as part of the wrong Action so I went looking for a logging middleware I could compare with.

I found redux_logging | Dart Package and noticed they said:

Note: The LoggingMiddleware needs be the LAST middleware in the list.

I moved my RemoteDevToolsMiddleware to the end of the middleware list and the problems were solved.

Is this a known issue? I can create a minimal repro if you'd like. Thanks!

MichaelMarner commented 4 years ago

Good pickup. We don't use many middleware packages in our production app, and so haven't been caught by this. Makes sense that we would have the same requirement as redux_logging though

MichaelMarner commented 4 years ago

Have updated the README with this information and published as 1.0.3.

Thanks again!