-
Notifications
You must be signed in to change notification settings - Fork 3k
Description
Please make our job easier by filling this template out to completion. If you're requesting a feature instead of reporting a bug, please feel free to skip the Environment and Reproducible Demo sections.
Description
I'm using vite to include this in a react-native-web project and rollup is leaving the require statements in the code without bundling the dependencies because you are mixing import and require statements in the same file. See import and require
Expected Behavior
I expect a library to stick with either require or import. Not both
Observed Behavior
Uncaught ReferenceError: require is not defined
Environment
Please run these commands in the project folder and fill in their results:
npm ls react-native-calendars
:npm ls react-native
:
Also specify:
- Device/emulator/simulator & OS version:
Reproducible Demo
Please provide a minimized reproducible demonstration of the problem you're reporting.
Issues that come with minimal repro's are resolved much more quickly than issues where a maintainer has to reproduce themselves.
Screenshots
Screenshots or gifs of the issue and the suggested fix will help us move faster with the review process.
Activity
kmalakoff commentedon Jan 18, 2024
Here's the patch file I used to move all the code to use import rather than mixed module systems in the same file:
stale commentedon Apr 22, 2024
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
fershibli commentedon May 8, 2024
Did they fix this? I'm getting a bunch of errors, so I'll give the patch a try
stale commentedon Feb 1, 2025
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.