Remove provide and Connector from public API

Author: gaearon

README.md

@@ -9,12 +9,9 @@ For React Native, import from `react-redux/native` instead.
 >**Note: There is a project called “redux-react” on NPM that is completely unrelated to the official bindings. This documentation (and any other official Redux documentation) is for `react-redux`.**
 
 - [Quick Start](#quick-start)
-- [Recommended API](#recommended-api)
+- [API](#api)
   - [`connect`](#connect)
   - [`Provider`](#provider)
-- [Deprecated API](#deprecated-api)
-  - [`Connector`](#connector)
-  - [`provide`](#provide)
 
 ## Quick Start
 
@@ -214,7 +211,7 @@ React.render((
 // );
 ```
 
-## Recommended API
+## API
 
 ### `connect`
 
@@ -246,42 +243,7 @@ See the usage example in the quick start above.
 
 The `Provider` component takes a `store` prop and a [function as a child](#child-must-be-a-function) with your root
 component. The `store` is then passed to the child via React's `context`. This is the entry point for Redux and must be
-present in order to use the `Connector` component.
-
-## Deprecated API
-
-### `Connector`
-
->**Note**  
->Deprecated. Use `connect()` instead.
-
-```js
-<Connector select={fn}>
-  {props => <MyComponent {...props}  />}
-</Connector>
-```
-
-Similar to `Provider`, the `Connector` expects a single [function as a child](#child-must-be-a-function) and a function
-as the `select` prop. The selector function takes a single argument of the entire root store and returns an object to be
-passed as properties to the child. In addition to the properties returned by the selector, a `dispatch` function is
-passed to the child for dispatching actions.
-
-It is the responsibility of a Smart Component to bind action creators to the given `dispatch` function and pass those
-bound creators to Dumb Components. Redux provides a `bindActionCreators` to streamline the process of binding action
-creators to the dispatch function.
-
-We don’t recommend its use anymore because it’s not as flexible as `connect()` and has some performance implications for more complex scenarios.
-
-### `provide`
-
->**Note**  
->Deprecated. Use `<Provider>` instead.
-
-```js
-export default provide(store)(MyRootComponent);
-```
-
-This higher-order component provides the same functionality as `<Provider>`. We don’t recommend it anymore because it’s less flexible than `<Provider>` and doesn’t work with [redux-devtools](http://github.com/gaearon/redux-devtools) or server rendering.
+present in order to use the `connect` component.
 
 ## License
 

src/components/createAll.js

@@ -1,17 +1,12 @@
 import createProvider from './createProvider';
-import createProvideDecorator from './createProvideDecorator';
 
 import createConnector from './createConnector';
 import createConnectDecorator from './createConnectDecorator';
 
 export default function createAll(React) {
-  // Wrapper components
   const Provider = createProvider(React);
-  const Connector = createConnector(React);
+  const connect = createConnectDecorator(React, createConnector(React));
 
-  // Higher-order components (decorators)
-  const provide = createProvideDecorator(React, Provider);
-  const connect = createConnectDecorator(React, Connector);
-
-  return { Provider, Connector, provide, connect };
+  // provider and Connector are deprecated and removed from public API
+  return { Provider, connect };
 }

src/components/createConnector.js

@@ -3,6 +3,7 @@ import shallowEqual from '../utils/shallowEqual';
 import isPlainObject from '../utils/isPlainObject';
 import invariant from 'invariant';
 
+// Connector is deprecated and removed from public API
 export default function createConnector(React) {
   const { Component, PropTypes } = React;
   const storeShape = createStoreShape(PropTypes);

src/components/createProvideDecorator.js

@@ -1,4 +1,5 @@
 import React from 'react';
 import createAll from './components/createAll';
 
-export const { Provider, Connector, provide, connect } = createAll(React);
+// provide and Connector are deprecated and removed from public API
+export const { Provider, connect } = createAll(React);

src/index.js

@@ -1,4 +1,5 @@
 import React from 'react-native';
 import createAll from './components/createAll';
 
-export const { Provider, Connector, provide, connect } = createAll(React);
+// provide and Connector are deprecated and removed from public API
+export const { Provider, connect } = createAll(React);

src/native.js

@@ -2,8 +2,9 @@ import expect from 'expect';
 import jsdomReact from './jsdomReact';
 import React, { PropTypes, Component } from 'react/addons';
 import { createStore } from 'redux';
-import { Connector } from '../../src/index';
+import createConnector from '../../src/components/createConnector';
 
+const Connector = createConnector(React);
 const { TestUtils } = React.addons;
 
 describe('React', () => {

test/components/Connector.spec.js

@@ -2,7 +2,7 @@ import expect from 'expect';
 import jsdomReact from './jsdomReact';
 import React, { PropTypes, Component } from 'react/addons';
 import { createStore } from 'redux';
-import { connect, Connector } from '../../src/index';
+import { connect } from '../../src/index';
 
 const { TestUtils } = React.addons;
 
@@ -45,9 +45,11 @@ describe('React', () => {
       const div = TestUtils.findRenderedDOMComponentWithTag(container, 'div');
       expect(div.props.pass).toEqual('through');
       expect(div.props.foo).toEqual('bar');
-      expect(() =>
-        TestUtils.findRenderedComponentWithType(container, Connector)
-      ).toNotThrow();
+
+      // Connector is deprecated and removed from public API
+      // expect(() =>
+      //   TestUtils.findRenderedComponentWithType(container, Connector)
+      // ).toNotThrow();
     });
 
     it('should handle additional prop changes in addition to slice', () => {
@@ -118,13 +120,15 @@ describe('React', () => {
           {() => <Container pass='through' />}
         </Provider>
       );
-      const connector = TestUtils.findRenderedComponentWithType(container, Connector);
-      expect(connector.props.select({
-        foo: 5,
-        bar: 7
-      })).toEqual({
-        foo: 5
-      });
+
+      // Connector is deprecated and removed from public API
+      // const connector = TestUtils.findRenderedComponentWithType(container, Connector);
+      // expect(connector.props.select({
+      //   foo: 5,
+      //   bar: 7
+      // })).toEqual({
+      //   foo: 5
+      // });
     });
 
     it('should set the displayName correctly', () => {