Skip to content

Commit

Permalink
Wordsmithing deprecation notices (and adding fb.me links)
Browse files Browse the repository at this point in the history
  • Loading branch information
Brian Vaughn committed Apr 11, 2017
1 parent 488c8d2 commit 83bcb29
Show file tree
Hide file tree
Showing 2 changed files with 26 additions and 12 deletions.
17 changes: 12 additions & 5 deletions src/isomorphic/React.js
Original file line number Diff line number Diff line change
Expand Up @@ -106,8 +106,11 @@ if (__DEV__) {
get() {
warning(
warnedForCheckPropTypes,
'checkPropTypes has moved out of the react package. ' +
'Use the prop-types package from npm instead.',
'checkPropTypes have been moved to a separate package. ' +
'Accessing React.checkPropTypes is no longer supported, ' +
'and will be removed completely in React 16. ' +
'Use the prop-types package on npm instead. ' +
'(https://fb.me/migrating-from-react-proptypes)',
);
warnedForCheckPropTypes = true;
return ReactPropTypes;
Expand All @@ -121,7 +124,8 @@ if (__DEV__) {
'React.createClass is no longer supported. Use a plain JavaScript ' +
"class instead. If you're not yet ready to migrate, " +
'create-react-class is available on npm as a temporary, ' +
'drop-in replacement.',
'drop-in replacement. ' +
'(https://fb.me/migrating-from-react-create-class)',
);
warnedForCreateClass = true;
return createReactClass;
Expand All @@ -132,8 +136,11 @@ if (__DEV__) {
get() {
warning(
warnedForPropTypes,
'PropTypes have moved out of the react package. ' +
'Use the prop-types package from npm instead.',
'PropTypes have been moved to a separate package. ' +
'Accessing React.PropTypes is no longer supported, ' +
'and will be removed completely in React 16. ' +
'Use the prop-types package on npm instead. ' +
'(https://fb.me/migrating-from-react-proptypes)',
);
warnedForPropTypes = true;
return ReactPropTypes;
Expand Down
21 changes: 14 additions & 7 deletions src/isomorphic/__tests__/React-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -35,10 +35,11 @@ describe('React', () => {
expect(createClass).not.toBe(undefined);
expectDev(console.error.calls.count()).toBe(1);
expectDev(console.error.calls.argsFor(0)[0]).toContain(
'React.createClass is no longer supported. Use a plain ' +
"JavaScript class instead. If you're not yet ready to migrate, " +
'React.createClass is no longer supported. Use a plain JavaScript ' +
"class instead. If you're not yet ready to migrate, " +
'create-react-class is available on npm as a temporary, ' +
'drop-in replacement.',
'drop-in replacement. ' +
'(https://fb.me/migrating-from-react-create-class)',
);
});

Expand All @@ -49,8 +50,11 @@ describe('React', () => {
expect(PropTypes).not.toBe(undefined);
expectDev(console.error.calls.count()).toBe(1);
expectDev(console.error.calls.argsFor(0)[0]).toContain(
'PropTypes have moved out of the react package. ' +
'Use the prop-types package from npm instead.',
'PropTypes have been moved to a separate package. ' +
'Accessing React.PropTypes is no longer supported, ' +
'and will be removed completely in React 16. ' +
'Use the prop-types package on npm instead. ' +
'(https://fb.me/migrating-from-react-proptypes)',
);
});

Expand All @@ -61,8 +65,11 @@ describe('React', () => {
expect(checkPropTypes).not.toBe(undefined);
expectDev(console.error.calls.count()).toBe(1);
expectDev(console.error.calls.argsFor(0)[0]).toContain(
'checkPropTypes has moved out of the react package. ' +
'Use the prop-types package from npm instead.',
'checkPropTypes have been moved to a separate package. ' +
'Accessing React.checkPropTypes is no longer supported, ' +
'and will be removed completely in React 16. ' +
'Use the prop-types package on npm instead. ' +
'(https://fb.me/migrating-from-react-proptypes)',
);
});
});

0 comments on commit 83bcb29

Please sign in to comment.