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

fix js build for pulldown #544

Closed
wants to merge 6 commits into from
Closed

fix js build for pulldown #544

wants to merge 6 commits into from

Conversation

mccalluc
Copy link
Contributor

but right now this doesn't work. (Errors with Uncaught ReferenceError: jquery is not defined.) Filing as draft, just so the branch doesn't get lost.

The most useful thing would probably be figuring out how to replicate the production build locally.
I've put more info on the history of the bug in the issue.

@pulbot pulbot temporarily deployed to staging October 21, 2022 21:36 Inactive
@mccalluc
Copy link
Contributor Author

Still not working:

Uncaught Error: rails-ujs has already been loaded!

@mccalluc
Copy link
Contributor Author

Closing. We realized that this error was caused by using the VPN with a prefixed URL. The issue should remain open, but it's less pressing.

@mccalluc mccalluc closed this Oct 28, 2022
@mccalluc mccalluc deleted the i540-js-build branch October 28, 2022 18:55
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: import declarations may only appear at top level of a module"
2 participants