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.

Sign up for GitHub

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

Jump to bottom

Get it to work with StackBlitz? #4

Closed
jcbhmr opened this issue Apr 13, 2023 · 6 comments
Closed

Get it to work with StackBlitz? #4

jcbhmr opened this issue Apr 13, 2023 · 6 comments
Assignees
@jcbhmr
Labels
bug Something isn't working wontfix This will not be worked on

Comments

@jcbhmr
Copy link
Member

jcbhmr commented Apr 13, 2023

image
image
image
@jcbhmr jcbhmr added bug Something isn't working help wanted Extra attention is needed question Further information is requested labels Apr 13, 2023
@jcbhmr jcbhmr added this to the v2 milestone Apr 13, 2023
@jcbhmr jcbhmr self-assigned this Apr 13, 2023
@jcbhmr
Copy link
Member Author

jcbhmr commented Apr 13, 2023

So I narrowed it down to this error

image

which seems to indicate that the import() function/magic/keyword thing isn't defined?

@jcbhmr
Copy link
Member Author

jcbhmr commented Apr 13, 2023

But then if I dynamic import anything, even in the subprocess.js file, it works a-OK

image
image

@jcbhmr
Copy link
Member Author

jcbhmr commented Apr 13, 2023 •
edited
Loading

OK so I remembered that eval() indirectly runs things in global CLASSIC SCRIPT MODE global scope. I'm guessing that import() isn't available there?

yeah this seems to be it!

image
image

now there's a different error, but it did fix that part!

@jcbhmr
Copy link
Member Author

jcbhmr commented Apr 13, 2023

image
ok so i guess i need to check the node version of stackblitz...

@jcbhmr
Copy link
Member Author

jcbhmr commented Apr 13, 2023 •
edited
Loading

image
that would be why...

stackblitz/webcontainer-core#560

@jcbhmr jcbhmr added wontfix This will not be worked on and removed help wanted Extra attention is needed question Further information is requested labels Apr 13, 2023
@jcbhmr
Copy link
Member Author

jcbhmr commented Apr 13, 2023

I added a note to the readme that this needs Node.js v17+

@jcbhmr jcbhmr closed this as not planned Won't fix, can't repro, duplicate, stale Apr 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jcbhmr jcbhmr

Labels
bug Something isn't working wontfix This will not be worked on
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jcbhmr