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

Add support for redux-devtools-extension #12

Closed
wants to merge 6 commits into from

Conversation

tdbrian
Copy link

@tdbrian tdbrian commented Aug 6, 2016

https://github.com/zalmoxisus/redux-devtools-extension is a very popular extension for Redux but because 'createStore' is not available outside the library for this extension, I went ahead and added support for the debugging extension at the create store. The tests succeeded after changes, and I tested that the extension does indeed work now with this library. Please let me know if you'd like me to make different changes to allow for this support.

Other commits include changing TSD to Typings which is recommended now by the TSD author, adding Store to index.d.ts, adding new scripts to package.json for Typings and running test on Windows, fixing issue where initial Redux action type was not being caught for setting initial state, and fixing typing errors in tests.

Thomas Brian added 6 commits August 6, 2016 16:02
@tdbrian
Copy link
Author

tdbrian commented Aug 7, 2016

Just noticed some of this work has already been done on the develop branch. I'm pulling the PR back and will reapply smaller PRs to the develop branch.

@tdbrian tdbrian closed this Aug 7, 2016
@KarlPurk
Copy link
Owner

KarlPurk commented Aug 8, 2016

Thanks for this @tdbrian - looking forward to the new PR.

@KarlPurk
Copy link
Owner

KarlPurk commented Aug 8, 2016

@tdbrian I've just pushed a few changes to develop, mainly the addition of the must.d.ts file to satisfy the Typescript compiler. I'm waiting on must to cut a new release, then I can remove this file for good.

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

Successfully merging this pull request may close these issues.

2 participants