Skip to content

redux action type including init/replace types #3582

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

Closed
wants to merge 1 commit into from
Closed

redux action type including init/replace types #3582

wants to merge 1 commit into from

Conversation

Mazuh
Copy link
Contributor

@Mazuh Mazuh commented Oct 8, 2019

related to #3580

not sure if it's worth it, but I was here studying typescript/redux and thought this might help :) idk

@Mazuh Mazuh changed the title redux action including init/replace types redux action type including init/replace types Oct 8, 2019
@netlify
Copy link

netlify bot commented Oct 8, 2019

Deploy preview for redux-docs ready!

Built with commit 7c89e4a

https://deploy-preview-3582--redux-docs.netlify.com

@markerikson
Copy link
Contributor

Appreciate the PR, but we explicitly don't want to expose the INIT action type. That's internal, and should not be directly depended on by user code.

Not quite sure what @timdorr meant by his comment, exactly.

@Mazuh
Copy link
Contributor Author

Mazuh commented Oct 8, 2019

I see. If it's intentional then there's no use for this. Closing it. Thanks for the quick feedback, tho.

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