aweary / aweary/react-is-deprecated

Doesn't work with new 'prop-types' library

Open
#6 2 comments 0 reactions 0 assignees View on GitHub
help wanted
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.