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

bug: update script attribute #185

Closed
wants to merge 1 commit into from
Closed

Conversation

MartinMa
Copy link

@MartinMa MartinMa commented Aug 8, 2019

Closes #184

- Summary
Apparently hugo outputs main.js, not app.js anymore. So I updated the template accordingly. See linked issue.

@MartinMa
Copy link
Author

MartinMa commented Aug 8, 2019

Actually, I have to revoke this pull request!

Now I have main.js twice locally:
image

And this in the production version:
image

I don't know what's going on.

@MartinMa MartinMa closed this Aug 8, 2019
@erquhart
Copy link
Contributor

erquhart commented Aug 9, 2019

You should be able to just remove that script tag from the footer, the scripts are injected here: https://github.com/netlify-templates/one-click-hugo-cms/blob/master/site/layouts/_default/baseof.html#L14

@MartinMa
Copy link
Author

@erquhart Thanks for the answer! Can I remove the file app.js altogether? Its contents don't seem to be compiled into the bundle. The entry point seems to be index.js.

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.

Uncaught SyntaxError: Unexpected token <
2 participants