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

issue 15: fixed panResponder reference #16

Closed

Conversation

eaglejs
Copy link

@eaglejs eaglejs commented Oct 4, 2020

Issue #15: This addresses an issue where if a developer is using TypeScript, the application will fail to build because panResponder is not defined. Adding it as a property to the class seems to solve this issue.

@eaglejs
Copy link
Author

eaglejs commented Oct 13, 2020

Sorry for the delay. So I tried to deploy it and I got this issue.

Failed building JavaScript bundle.
While trying to resolve module `react-native-circle-slider` from file `./screens/dashboard.screen.tsx`, the package `./node_modules/react-native-circle-slider/package.json` was successfully found. However, this package itself specifies a `main` module field that could not be resolved (`./node_modules/react-native-circle-slider/index.js`. Indeed, none of these files exist:

  * ./node_modules/react-native-circle-slider/index.js(.native|.android.expo.ts|.native.expo.ts|.expo.ts|.android.expo.tsx|.native.expo.tsx|.expo.tsx|.android.expo.js|.native.expo.js|.expo.js|.android.expo.jsx|.native.expo.jsx|.expo.jsx|.android.ts|.native.ts|.ts|.android.tsx|.native.tsx|.tsx|.android.js|.native.js|.js|.android.jsx|.native.jsx|.jsx|.android.json|.native.json|.json|.android.wasm|.native.wasm|.wasm)
  * ./node_modules/react-native-circle-slider/index.js/index(.native|.android.expo.ts|.native.expo.ts|.expo.ts|.android.expo.tsx|.native.expo.tsx|.expo.tsx|.android.expo.js|.native.expo.js|.expo.js|.android.expo.jsx|.native.expo.jsx|.expo.jsx|.android.ts|.native.ts|.ts|.android.tsx|.native.tsx|.tsx|.android.js|.native.js|.js|.android.jsx|.native.jsx|.jsx|.android.json|.native.json|.json|.android.wasm|.native.wasm|.wasm)

@raymondchooi
Copy link
Owner

Ah oops, thanks for checking, I've updated the main to use index.ts now

@eaglejs eaglejs closed this Oct 4, 2022
@eaglejs eaglejs force-pushed the issue-15-fix-panResponder-variable branch from 5992d55 to 94c6c58 Compare October 4, 2022 22:14
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.

None yet

2 participants