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: ensure package works as a module #1293

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jsakas
Copy link

@jsakas jsakas commented Apr 14, 2023

What kind of change does this PR introduce?

  • Bug Fix
  • Feature
  • Refactoring
  • Style
  • Build
  • Chore
  • Documentation
  • CI

Did you add tests for your changes?

  • Yes, my code is well tested
  • Not relevant

If relevant, did you update the documentation?

  • Yes, I've updated the documentation
  • Not relevant

Summary

This package is not compatible with ESM due to attr-accept default export.

As such, we are unable to use react-dropzone in an app powered by Vite without this change.

Closes #1259

Does this PR introduce a breaking change?

No

Other information

@JClackett
Copy link

please merge!

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.

[BUG] react-dropzone cannot be imported in a Node.js ESM environment
2 participants