Skip to content

Fixes error undefined is not an object (evaluating '_react2.PropTypes.string')#36

Open
crim3hound wants to merge 1 commit into
jaysoo:masterfrom
crim3hound:master
Open

Fixes error undefined is not an object (evaluating '_react2.PropTypes.string')#36
crim3hound wants to merge 1 commit into
jaysoo:masterfrom
crim3hound:master

Conversation

@crim3hound

Copy link
Copy Markdown

Fixes error undefined is not an object (evaluating '_react2.PropTypes.string').
React Native Version: 0.51.0

Fixes error `undefined is not an object (evaluating '_react2.PropTypes.string')`.
React Native Version: 0.51.0
@xiamx

xiamx commented Mar 9, 2018

Copy link
Copy Markdown

Do you want to rename the pull request title to "Fixes error undefined is not an object (evaluating '_react2.PropTypes.string')."? It would help other people finding this fix easily

@crim3hound crim3hound changed the title Update Prompt.js Fixes error undefined is not an object (evaluating '_react2.PropTypes.string') Mar 9, 2018
@perry-mitchell

perry-mitchell commented Mar 12, 2018

Copy link
Copy Markdown

This is excellent, thank you @anti-nerd - One problem though, it seems that this library does not yet reference prop-types in package.json.. Could you possibly add it? (not sure if it's suitable just as a peer dependency..) If not, react-native projects that only use basic components and no prop-types will throw errors I'm guessing. Having it as a peer dependency would at least warn them.

@perry-mitchell

Copy link
Copy Markdown

@jaysoo I'm sorry to pester you, but would you have an ETA for when this might be merged and released?

@crim3hound

crim3hound commented Mar 12, 2018

Copy link
Copy Markdown
Author

You're welcome @perry-mitchell, and I believe that's true since it seems that it's newer versions of RN that have a separate prop-types library. Let me look into that.

Alternatively, perhaps @jaysoo can update the library to use prop-types as one of the dependencies instead? This way the package.json would also be updated.

@perry-mitchell

perry-mitchell commented Mar 12, 2018

Copy link
Copy Markdown

I've released a fork of this repo under @perrymitchell/react-native-prompt, in case it might temporarily help someone, as I've needed to continue with other features and couldn't wait for this PR to be merged.

@DikshaDeep

Copy link
Copy Markdown

Please update the package and close this issue

@perry-mitchell

Copy link
Copy Markdown

@jaysoo Any chance of a quick merge and release? Seems like a really harmless PR. I would quickly close my fork in favour of this feature going out..

@xlogix

xlogix commented Feb 23, 2020

Copy link
Copy Markdown

Is this merged?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants