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

Using a name withe kebab case breaks server and MergeRuntime. #21

Closed
hamatoyogi opened this issue Nov 4, 2020 · 1 comment
Closed

Comments

@hamatoyogi
Copy link

Followed through @jherr 's video

Called my consumer nextjs app next-app-2.

After following instructions and running the consumer app I got a blank screen.

fiddled around with _document.js and next.config.js - commenting out stuff here and there which got it working.

re-added execution of MergeRuntime in config file and recreated blank screen.

Opened source files and main.js... next.js static chunk and saw that the eval got broken by the next-app-2 name:

image

Changing the name in the config file to a camelCase fixed the issue.

Could be good to add protection from this, or add documentation not to use that sort of app name (at least in the next.config.js)

@ScriptedAlchemy
Copy link
Member

not a valid variable... works as expected. You can set library type to window, instead of the default var which should make it work with dashed vars

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

2 participants