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

Remove seemingly incorrect/unneeded "type": "module" field #35

Merged
merged 1 commit into from
Jun 14, 2023

Conversation

sjdemartini
Copy link
Owner

By removing this, the package will default to being a CJS package. Other packages (e.g., tss-react, mui-datatables, MUI, react-pdf) do not specify this either.

From testing installing in a third-party application, this fixes an issue in testing code that imports mui-tiptap (at least when using vitest):

Error: Cannot find module '/path/node_modules/mui-tiptap/dist/esm/ControlledBubbleMenu' imported from /path/node_modules/mui-tiptap/dist/esm/index.js

By removing this, the package will default to being a CJS package. Other
packages (e.g., tss-react, mui-datatables, MUI, react-pdf) do not
specify this either.

From testing installing in a third-party application, this fixes an
issue in testing code that imports `mui-tiptap` (at least when using
vitest):

```
Error: Cannot find module '/path/node_modules/mui-tiptap/dist/esm/ControlledBubbleMenu' imported from /path/node_modules/mui-tiptap/dist/esm/index.js
```
@sjdemartini sjdemartini merged commit 9487384 into main Jun 14, 2023
@sjdemartini sjdemartini deleted the remove-type-module branch June 14, 2023 06:59
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.

1 participant