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

default export from babel-helper-vue-jsx-merge-props #148

Open
quinnlangille opened this issue Aug 30, 2018 · 4 comments
Open

default export from babel-helper-vue-jsx-merge-props #148

quinnlangille opened this issue Aug 30, 2018 · 4 comments

Comments

@quinnlangille
Copy link

I'm not sure if this is an issue for babel-helper-vue-jsx-merge-props or for here, but in the current version of vue-jsx we're adding a default import as import _mergeJSXProps from 'babel-helper-vue-jsx-merge-props'.

However, it doesn't seem that babel-helper-vue-jsx-merge-props has a default export. It's currently causing an error in my build. I can submit a PR with my fix, but I'm not sure what side effects it would have.

If you think it will be harmless to the vue-babel ecosystem then lemme know and I'll push it up :octocat:

@enjkvbej
Copy link

@quinnlangille hi how you resolved?

@pecliu
Copy link

pecliu commented Oct 25, 2021

I'm not sure if this is an issue for babel-helper-vue-jsx-merge-props or for here, but in the current version of vue-jsx we're adding a default import as import _mergeJSXProps from 'babel-helper-vue-jsx-merge-props'.

However, it doesn't seem that babel-helper-vue-jsx-merge-props has a default export. It's currently causing an error in my build. I can submit a PR with my fix, but I'm not sure what side effects it would have.

If you think it will be harmless to the vue-babel ecosystem then lemme know and I'll push it up :octocat:

@quinnlangille hi how you resolved?

@pecliu
Copy link

pecliu commented Oct 25, 2021

when I use vite & vue2.x & jsx, there came out this error

uncaught SyntaxError: The requested module '/node_modules/@vue/babel-helper-vue-jsx-merge-props/dist/helper.js?v=a5e8dfdf' does not provide an export named 'default'

@quinnlangille
Copy link
Author

Hey @pecliu, this was for an old PoC repo I was working on. I don't remember the exact fix, but I know I had to fork a bunch of packages and manually resolve their dependencies to get them to build. Here's the project if you want to take a look -> https://github.com/SSENSE/flip, if you check inside src/utils there are a few packages I've cloned and modified to get around the build error.

Not very helpful if you need to ship to production lol but if you're working locally that should +/- work

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

No branches or pull requests

3 participants