-
-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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 createProvider Documentation. #752
Conversation
@@ -384,3 +385,41 @@ function selectorFactory(dispatch) { | |||
export default connectAdvanced(selectorFactory)(TodoApp) | |||
``` | |||
|
|||
<a id="createProvider"></a> | |||
### `createProvider([storeKey])` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wanted to add subKey as well, but then over here:
https://github.com/reactjs/react-redux/blob/a9bb9d3b074eb04f4d7922e22fce733beb148650/src/components/connectAdvanced.js#L78
It is hardcoded to be
const subscriptionKey = storeKey + 'Subscription'
So I don't think I could pass it in connect
so it was broken anyways
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably should fix that...
LGTM. Thanks! |
* Add createProvider docs * Add storeKey to connect options * italics * Update api.md * Add you probably only need this
* Add createProvider docs * Add storeKey to connect options * italics * Update api.md * Add you probably only need this
Fixes #738
Open to Grammar, Capitalization & Style changes. Also can squash, rebase etc.