Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix PropTypes already moved to separate package #4

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

JiboStore
Copy link

Hi, PropTypes has moved to a separate package and no longer in react.

Your code hasn't been updated to accomodate this thus all the npm install for current projects will cause crashing when you access PropTypes

I've fixed this issue but when I do npm install I've got empty package underneath react-touch-position even though I point directly to my repo. I've never published npm package before and don't know the workflow, I just removed the dist/ under .gitignore and submitted the built-code underneath which makes it 'works' but I don't think it is the best idea. It works for me.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 82362b1 on JiboStore:master into 1133bca on ethanselzer:master.

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.

3 participants