container.training icon indicating copy to clipboard operation
container.training copied to clipboard

`$` Not defined

Open hebronwatson opened this issue 4 years ago • 3 comments

Summary

The timeseries in the Dockercoins Web App is non-functional

Probable Cause

Firefox does not recognize the contents of the jquery.js file as a symlink, and trying (unsuccessfully) to parse the contents (simply, jquery-1.11.3.min.js) as JavaScript.

Environment

Docker for Desktop on Windows 10

hebronwatson avatar Sep 16 '21 07:09 hebronwatson

Hi, thanks for this bug report!

If you edit index.html to update the link (putting jquery-1.11.3.min.js instead of jquery.js), does it fix the problem?

Thank you!

jpetazzo avatar Sep 16 '21 10:09 jpetazzo

Wow! THE JPetazzo is responding to my issue? Ok!

Yeah, it totally fixes it, but I was wondering why it's in there in the first place? Is it supposed to be a type of symlink, and if so, why does it not work? It really shouldn't be my environment that's the problem, right?

hebronwatson avatar Sep 16 '21 17:09 hebronwatson

Hi!

I think I made this to be able to easily change the version of jquery by updating a symlink, instead of updating the HTML code. When I wrote the app, I thought I might end up with multiple HTML pages, so I wanted to be able to switch dependencies in a single place. I'm not 100% sure though, because I wrote this thing 6 years ago and I had never needed to look again at that particular piece of the code 😅

That said, I tried to reproduce the bug on my Windows 10 machine, and ... it worked fine 🤔 On my machine, the jquery.js symlink shows up as <JUNCTION> when I run dir. Out of curiosity, what does it look like on your machine? And are you using WSL2? (No idea if it matters or not, but I'm using it, because this is a Windows 10 Home machine.)

Either way, I think I'll edit the index.html file after all; but I really wonder what's going on there :)

jpetazzo avatar Oct 04 '21 05:10 jpetazzo