Skip to content

Commit 7783f88

Browse files
yungstersfacebook-github-bot
authored andcommitted
Revise Prop Types Error Message (#34392)
Summary: Improves upon the existing prop types error messages to first recommend migrating to a type system (before recommending the usage of `deprecated-react-native-prop-types`). ## Changelog [General][Changed] - Minor change to PropTypes error message. Pull Request resolved: #34392 Test Plan: Careful code inspection. Reviewed By: rickhanlonii Differential Revision: D38632723 Pulled By: yungsters fbshipit-source-id: 88e44116475c7d93bc561c9ab6883855d6d79a7e
1 parent ffaa1a6 commit 7783f88

File tree

1 file changed

+20
-8
lines changed

1 file changed

+20
-8
lines changed

index.js

Lines changed: 20 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -427,29 +427,41 @@ module.exports = {
427427
get ColorPropType(): $FlowFixMe {
428428
invariant(
429429
false,
430-
'ColorPropType has been removed from React Native. Migrate to ' +
431-
"ColorPropType exported from 'deprecated-react-native-prop-types'.",
430+
'ColorPropType has been removed from React Native, along with all ' +
431+
'other PropTypes. We recommend that you migrate away from PropTypes ' +
432+
'and switch to a type system like TypeScript. If you need to ' +
433+
'continue using ColorPropType, migrate to the ' +
434+
"'deprecated-react-native-prop-types' package.",
432435
);
433436
},
434437
get EdgeInsetsPropType(): $FlowFixMe {
435438
invariant(
436439
false,
437-
'EdgeInsetsPropType has been removed from React Native. Migrate to ' +
438-
"EdgeInsetsPropType exported from 'deprecated-react-native-prop-types'.",
440+
'EdgeInsetsPropType has been removed from React Native, along with all ' +
441+
'other PropTypes. We recommend that you migrate away from PropTypes ' +
442+
'and switch to a type system like TypeScript. If you need to ' +
443+
'continue using EdgeInsetsPropType, migrate to the ' +
444+
"'deprecated-react-native-prop-types' package.",
439445
);
440446
},
441447
get PointPropType(): $FlowFixMe {
442448
invariant(
443449
false,
444-
'PointPropType has been removed from React Native. Migrate to ' +
445-
"PointPropType exported from 'deprecated-react-native-prop-types'.",
450+
'PointPropType has been removed from React Native, along with all ' +
451+
'other PropTypes. We recommend that you migrate away from PropTypes ' +
452+
'and switch to a type system like TypeScript. If you need to ' +
453+
'continue using PointPropType, migrate to the ' +
454+
"'deprecated-react-native-prop-types' package.",
446455
);
447456
},
448457
get ViewPropTypes(): $FlowFixMe {
449458
invariant(
450459
false,
451-
'ViewPropTypes has been removed from React Native. Migrate to ' +
452-
"ViewPropTypes exported from 'deprecated-react-native-prop-types'.",
460+
'ViewPropTypes has been removed from React Native, along with all ' +
461+
'other PropTypes. We recommend that you migrate away from PropTypes ' +
462+
'and switch to a type system like TypeScript. If you need to ' +
463+
'continue using ViewPropTypes, migrate to the ' +
464+
"'deprecated-react-native-prop-types' package.",
453465
);
454466
},
455467
};

0 commit comments

Comments
 (0)