aweary / aweary/react-is-deprecated
Doesn't work with new 'prop-types' library
- Dominant language
- JavaScript
- Stars
- 44
- Forks
- 6
- PR merge metrics
- No merged PRs in 30d
Description
In the addIsDeprecated wrapper you iterate over all properties on the PropTypes object and call `.bind()` for each property. However, in the new prop-types library they added a self-reference of `PropTypes` into the PropTypes object which causes it to break since it's not a function. There is also a `checkPropTypes` which may also be problematic.
You can view their source [here](https://github.com/facebook/prop-types/blob/master/factoryWithTypeCheckers.js)
Contributor guide
No contributing guide indexed for this repository
Research direction
Start in the addIsDeprecated wrapper and compare its iteration over PropTypes with the linked prop-types source, including the self-reference and checkPropTypes entries. Confirm the failure case, then verify that the wrapper handles non-function properties without breaking existing PropTypes behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, react
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100