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

Replay Actions #173

Closed
ccorcos opened this issue Aug 1, 2016 · 6 comments
Closed

Replay Actions #173

ccorcos opened this issue Aug 1, 2016 · 6 comments

Comments

@ccorcos
Copy link

ccorcos commented Aug 1, 2016

it would be great if we could import an array of actions to pull up a slider and replay the app. the states should be reproducible and this way, we could save tracking data thats captured from elsewhere and replay exactly what happened.

I was poking around and it looks like we'd need to add something here:

https://github.com/zalmoxisus/redux-devtools-extension/blob/master/src/app/api/importState.js

but im not sure how it would really work. I think we just want to reset the app, then dispatch all the actions in a row to generate the states. then we can go straight into the replay mode... what do you think?

@zalmoxisus
Copy link
Owner

Actions are already replayed (states are recomputed), so it should be easy to implement. It would be indeed useful, if you could send a PR here. We could just add this implementation if Array.isArray(nextLiftedState) for IMPORT_STATE action, then there's no need to change anything on the extension's side. Also please add tests.

Just curious form where you export an array of actions?

@ccorcos
Copy link
Author

ccorcos commented Aug 2, 2016

I created some middleware that sends all user actions to elasticsearch so we can do user analytics. I can query for a session, export all the redux actions into an array, checkout the proper release of the application, and replay the user's experience. :)

@ccorcos
Copy link
Author

ccorcos commented Aug 2, 2016

I'll get on that PR soon. Thanks for pointing me in the right direction :)

@zalmoxisus
Copy link
Owner

Hey @ccorcos,

Did you have a chance to check if that works? Those changes should be merged in the extension last week.

@ccorcos
Copy link
Author

ccorcos commented Aug 15, 2016

Not yet -- will do soon though.

@zalmoxisus
Copy link
Owner

@ccorcos, BTW, @timarney already tried it and even implemented in a lib.

@ccorcos ccorcos closed this as completed Aug 29, 2016
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

2 participants