Use prop-types package instead of React.PropTypes #122
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: petetnt <[email protected]>
Codecov Report
@@ Coverage Diff @@
## v1.4.0 #122 +/- ##
=========================================
Coverage ? 89.06%
=========================================
Files ? 17
Lines ? 1372
Branches ? 225
=========================================
Hits ? 1222
Misses ? 150
Partials ? 0 |
|
Great catch! Planned to be released as part of |
prescottprue
pushed a commit
that referenced
this pull request
May 17, 2017
# Conflicts: # docs/recipes/auth.md # docs/roadmap.md # examples/complete/material/src/routes/Home/components/NewTodoPanel/NewTo doPanel.js # examples/complete/material/src/routes/Home/components/TodoItem/TodoItem. js # examples/complete/material/src/routes/Home/containers/HomeContainer.js # examples/complete/material/src/routes/Login/components/LoginForm/LoginFo rm.js # examples/complete/material/src/routes/Projects/components/NewProjectDial og/NewProjectDialog.js # examples/complete/material/src/routes/Projects/containers/ProjectsContai ner.js # examples/complete/material/src/routes/Projects/routes/Project/components /Project/Project.js # examples/complete/material/src/routes/Recover/components/EmailForm/Email Form.js # examples/complete/material/src/routes/Recover/components/RecoverForm/Rec overForm.js # examples/complete/material/src/routes/Recover/containers/RecoverContaine r.js # examples/complete/simple/src/Home.js # examples/complete/simple/src/TodoItem.js # examples/snippets/decorators/App.js # examples/snippets/decorators/TodoItem.js # examples/snippets/multipleQueries/App.js # examples/snippets/multipleQueries/TodoItem.js # examples/snippets/populates/App.js # examples/snippets/stateBasedQuery/App.js # package.json # src/connect.js # tests/unit/connect.spec.js
prescottprue
added a commit
that referenced
this pull request
May 23, 2017
### Description * Adds `updateProfile`, `updateAuth`, and `updateEmail` methods that dispatch associated start/success/failure actions * Adds multiple features to populate - #132 * Uses `prop-types` instead of importing from react - #122 * `distpatchOnUnsetListener` fixed to be `dispatchOnUnsetListener` (depreciation warning added along with a test confirm it is displayed) * Do not include `dbPath` in response from `deleteFile` method if it is undefined (test added to check this case) * `.eslintrc` file now using yaml format instead of JSON format (easier to read) * Unnessesary global eslint comments removed from tests (no longer needed due to globals being moved to `.eslintrc`) * `enableEmptyAuthChanges` config option added - #137 ### Check List - [X] All tests passing - [X] Docs updated with any changes or examples - [X] Added tests to ensure feature(s) work properly ### Relevant Issues * [#122](https://github.com/prescottprue/react-redux-firebase/issues/#122) * [#132](https://github.com/prescottprue/react-redux-firebase/issues/#132) * [#137](https://github.com/prescottprue/react-redux-firebase/issues/#137)
Merged
prescottprue
added a commit
that referenced
this pull request
Sep 18, 2017
🍾 🥂 The long awaited v1.5.0 release 🥂 🍾 ### Breaking Changes * `browser` field has been removed from `package.json` which means commonJS build is used instead of `UMD` build from Webpack (much smaller) * [firebaseConnect] Uses `prop-types` instead of importing from React - #122 ### Core * [firebaseConnect] Allow usage of different stores in `firebaseConnect` - #148, #29 * [firebaseConnect] statics are now hoisted thanks to `hoist-non-react-statics` * [firebaseConnect] exposed `createFirebaseConnect` for creating `firebaseConnect` HOCs connected to different stores * [auth] `login` method supports `credential` parameter (with matching docs and tests updates) * [auth] deprecation warning added for `token` and `provider` combo in `login` method * [auth] Adds `updateProfile`, `updateAuth`, and `updateEmail` methods that dispatch associated start/success/failure actions * [helpers] `deleteFile` method response no longer includes `dbPath` if it is undefined (test added to check this case) * [helpers] No more race condition with `uniqueSet` - #207 * [typescript] Typescript typings - #142, #214 * [query] `watchEvent` accepts options object as third argument * [populate] Lodash path syntax support - #132 * [populate] accept a function for populate for item based populate config - #132 * [populate] Profile population (`profileParamsToPopulate`) supports single item and list population - #203 * [storage] `uploadFile` and `uploadFiles` no longer track progress by default (`storageRef.put` called directly) * [config] Usage of `profileFactory` is wrapped in try/catch to handle and reject errors within provided factory function * [config] `distpatchOnUnsetListener` spelling fixed to be `dispatchOnUnsetListener` (depreciation warning added along with a test confirm it is displayed) * [config] `enableEmptyAuthChanges` config option added - #137 ### Build * Firebase is now an external in Webpack config (shrinks UMD build size) * `.npmignore` expanded to include more files that do not needed to be included in published version (including `.babelrc` which can cause build errors) * `.babelrc` settings are now environment specific (decorators only used in testing) * `babel-preset-stage-1` removed (unnecessary) * `.eslintrc` added to tests folder which contains globals that are only necessary for testing (simplifies top level eslint config) * `.eslintrc` file now using yaml format instead of JSON format (easier to read) * `async/await` added to unit tests * Webpack 2 * [`firebase-server`](https://github.com/urish/firebase-server) used in tests instead of connecting to a real Firebase instance ### Docs * Query docs updated with `storeAs` and `keyProp` query options * `redux-persist` recipe added
prescottprue
added a commit
that referenced
this pull request
Sep 18, 2017
🍾 🥂 The long awaited v1.5.0 release 🥂 🍾 * `browser` field has been removed from `package.json` which means commonJS build is used instead of `UMD` build from Webpack (much smaller) * [firebaseConnect] Uses `prop-types` instead of importing from React - #122 * [firebaseConnect] Allow usage of different stores in `firebaseConnect` - #148, #29 * [firebaseConnect] statics are now hoisted thanks to `hoist-non-react-statics` * [firebaseConnect] exposed `createFirebaseConnect` for creating `firebaseConnect` HOCs connected to different stores * [auth] `login` method supports `credential` parameter (with matching docs and tests updates) * [auth] deprecation warning added for `token` and `provider` combo in `login` method * [auth] Adds `updateProfile`, `updateAuth`, and `updateEmail` methods that dispatch associated start/success/failure actions * [helpers] `deleteFile` method response no longer includes `dbPath` if it is undefined (test added to check this case) * [helpers] No more race condition with `uniqueSet` - #207 * [typescript] Typescript typings - #142, #214 * [query] `watchEvent` accepts options object as third argument * [populate] Lodash path syntax support - #132 * [populate] accept a function for populate for item based populate config - #132 * [populate] Profile population (`profileParamsToPopulate`) supports single item and list population - #203 * [storage] `uploadFile` and `uploadFiles` no longer track progress by default (`storageRef.put` called directly) * [config] Usage of `profileFactory` is wrapped in try/catch to handle and reject errors within provided factory function * [config] `distpatchOnUnsetListener` spelling fixed to be `dispatchOnUnsetListener` (depreciation warning added along with a test confirm it is displayed) * [config] `enableEmptyAuthChanges` config option added - #137 * Firebase is now an external in Webpack config (shrinks UMD build size) * `.npmignore` expanded to include more files that do not needed to be included in published version (including `.babelrc` which can cause build errors) * `.babelrc` settings are now environment specific (decorators only used in testing) * `babel-preset-stage-1` removed (unnecessary) * `.eslintrc` added to tests folder which contains globals that are only necessary for testing (simplifies top level eslint config) * `.eslintrc` file now using yaml format instead of JSON format (easier to read) * `async/await` added to unit tests * Webpack 2 * [`firebase-server`](https://github.com/urish/firebase-server) used in tests instead of connecting to a real Firebase instance * Query docs updated with `storeAs` and `keyProp` query options * `redux-persist` recipe added
prescottprue
added a commit
that referenced
this pull request
Sep 18, 2017
🍾 🥂 The long awaited v1.5.0 release 🥂 🍾 * `browser` field has been removed from `package.json` which means commonJS build is used instead of `UMD` build from Webpack (much smaller) * [firebaseConnect] Uses `prop-types` instead of importing from React - #122 * [firebaseConnect] Allow usage of different stores in `firebaseConnect` - #148, #29 * [firebaseConnect] statics are now hoisted thanks to `hoist-non-react-statics` * [firebaseConnect] exposed `createFirebaseConnect` for creating `firebaseConnect` HOCs connected to different stores * [auth] `login` method supports `credential` parameter (with matching docs and tests updates) * [auth] deprecation warning added for `token` and `provider` combo in `login` method * [auth] Adds `updateProfile`, `updateAuth`, and `updateEmail` methods that dispatch associated start/success/failure actions * [helpers] `deleteFile` method response no longer includes `dbPath` if it is undefined (test added to check this case) * [helpers] No more race condition with `uniqueSet` - #207 * [typescript] Typescript typings - #142, #214 * [query] `watchEvent` accepts options object as third argument * [populate] Lodash path syntax support - #132 * [populate] accept a function for populate for item based populate config - #132 * [populate] Profile population (`profileParamsToPopulate`) supports single item and list population - #203 * [storage] `uploadFile` and `uploadFiles` no longer track progress by default (`storageRef.put` called directly) * [config] Usage of `profileFactory` is wrapped in try/catch to handle and reject errors within provided factory function * [config] `distpatchOnUnsetListener` spelling fixed to be `dispatchOnUnsetListener` (depreciation warning added along with a test confirm it is displayed) * [config] `enableEmptyAuthChanges` config option added - #137 * Firebase is now an external in Webpack config (shrinks UMD build size) * `.npmignore` expanded to include more files that do not needed to be included in published version (including `.babelrc` which can cause build errors) * `.babelrc` settings are now environment specific (decorators only used in testing) * `babel-preset-stage-1` removed (unnecessary) * `.eslintrc` added to tests folder which contains globals that are only necessary for testing (simplifies top level eslint config) * `.eslintrc` file now using yaml format instead of JSON format (easier to read) * `async/await` added to unit tests * Webpack 2 * [`firebase-server`](https://github.com/urish/firebase-server) used in tests instead of connecting to a real Firebase instance * Query docs updated with `storeAs` and `keyProp` query options * `redux-persist` recipe added
prescottprue
added a commit
that referenced
this pull request
Sep 18, 2017
🍾 🥂 The long awaited v1.5.0 release 🥂 🍾 * `browser` field has been removed from `package.json` which means commonJS build is used instead of `UMD` build from Webpack (much smaller) * [firebaseConnect] Uses `prop-types` instead of importing from React - #122 * [firebaseConnect] Allow usage of different stores in `firebaseConnect` - #148, #29 * [firebaseConnect] statics are now hoisted thanks to `hoist-non-react-statics` * [firebaseConnect] exposed `createFirebaseConnect` for creating `firebaseConnect` HOCs connected to different stores * [auth] `login` method supports `credential` parameter (with matching docs and tests updates) * [auth] deprecation warning added for `token` and `provider` combo in `login` method * [auth] Adds `updateProfile`, `updateAuth`, and `updateEmail` methods that dispatch associated start/success/failure actions * [helpers] `deleteFile` method response no longer includes `dbPath` if it is undefined (test added to check this case) * [helpers] No more race condition with `uniqueSet` - #207 * [typescript] Typescript typings - #142, #214 * [query] `watchEvent` accepts options object as third argument * [populate] Lodash path syntax support - #132 * [populate] accept a function for populate for item based populate config - #132 * [populate] Profile population (`profileParamsToPopulate`) supports single item and list population - #203 * [storage] `uploadFile` and `uploadFiles` no longer track progress by default (`storageRef.put` called directly) * [config] Usage of `profileFactory` is wrapped in try/catch to handle and reject errors within provided factory function * [config] `distpatchOnUnsetListener` spelling fixed to be `dispatchOnUnsetListener` (depreciation warning added along with a test confirm it is displayed) * [config] `enableEmptyAuthChanges` config option added - #137 * Firebase is now an external in Webpack config (shrinks UMD build size) * `.npmignore` expanded to include more files that do not needed to be included in published version (including `.babelrc` which can cause build errors) * `.babelrc` settings are now environment specific (decorators only used in testing) * `babel-preset-stage-1` removed (unnecessary) * `.eslintrc` added to tests folder which contains globals that are only necessary for testing (simplifies top level eslint config) * `.eslintrc` file now using yaml format instead of JSON format (easier to read) * `async/await` added to unit tests * Webpack 2 * [`firebase-server`](https://github.com/urish/firebase-server) used in tests instead of connecting to a real Firebase instance * Query docs updated with `storeAs` and `keyProp` query options * `redux-persist` recipe added
prescottprue
added a commit
that referenced
this pull request
Sep 18, 2017
🍾 🥂 The long awaited v1.5.0 release 🥂 🍾 * `browser` field has been removed from `package.json` which means commonJS build is used instead of `UMD` build from Webpack (much smaller) * [firebaseConnect] Uses `prop-types` instead of importing from React - #122 * [firebaseConnect] Allow usage of different stores in `firebaseConnect` - #148, #29 * [firebaseConnect] statics are now hoisted thanks to `hoist-non-react-statics` * [firebaseConnect] exposed `createFirebaseConnect` for creating `firebaseConnect` HOCs connected to different stores * [auth] `login` method supports `credential` parameter (with matching docs and tests updates) * [auth] deprecation warning added for `token` and `provider` combo in `login` method * [auth] Adds `updateProfile`, `updateAuth`, and `updateEmail` methods that dispatch associated start/success/failure actions * [helpers] `deleteFile` method response no longer includes `dbPath` if it is undefined (test added to check this case) * [helpers] No more race condition with `uniqueSet` - #207 * [typescript] Typescript typings - #142, #214 * [query] `watchEvent` accepts options object as third argument * [populate] Lodash path syntax support - #132 * [populate] accept a function for populate for item based populate config - #132 * [populate] Profile population (`profileParamsToPopulate`) supports single item and list population - #203 * [storage] `uploadFile` and `uploadFiles` no longer track progress by default (`storageRef.put` called directly) * [config] Usage of `profileFactory` is wrapped in try/catch to handle and reject errors within provided factory function * [config] `distpatchOnUnsetListener` spelling fixed to be `dispatchOnUnsetListener` (depreciation warning added along with a test confirm it is displayed) * [config] `enableEmptyAuthChanges` config option added - #137 * Firebase is now an external in Webpack config (shrinks UMD build size) * `.npmignore` expanded to include more files that do not needed to be included in published version (including `.babelrc` which can cause build errors) * `.babelrc` settings are now environment specific (decorators only used in testing) * `babel-preset-stage-1` removed (unnecessary) * `.eslintrc` added to tests folder which contains globals that are only necessary for testing (simplifies top level eslint config) * `.eslintrc` file now using yaml format instead of JSON format (easier to read) * `async/await` added to unit tests * Webpack 2 * [`firebase-server`](https://github.com/urish/firebase-server) used in tests instead of connecting to a real Firebase instance * Query docs updated with `storeAs` and `keyProp` query options * `redux-persist` recipe added
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Signed-off-by: petetnt [email protected]
Description
The
React.PropTypeshave been deprecated and moved toprop-typespackage. Replaces instances ofReact.PropTypeswith theprop-typespackage to remove deprecation warnings from current users.For more information see https://facebook.github.io/react/warnings/dont-call-proptypes.html
Check List