-
Notifications
You must be signed in to change notification settings - Fork 133
JavaScript errors on page load #52
Comments
@marcesher It sounds like it might not be finding or loading https://github.com/etsy/morgue/blob/master/assets/js/jquery.timeentry-fr.js Can you be more specific about "when I load a morgue screen" - is that the home screen, the create or edit screens, or just everything? |
@cyberrodent I get that error above on both the main List screen and also the Edit screen. I see no 404 errors in dev tools or the apache log, but I also do not see the timeentry-fr.js file being pulled in. The only timeentry file I see is jquery.timeentry.min.js and timehelpers.js Here's the result from the apache access log for a single page request:
|
Oops - jquery.timeentry.min.js is indeed the file we care about. Sorry for the mix up. The error you are getting may indicate that you're not able to load jquery. This isn't bundled with the morge code, we're trying to pull it in from https://ajax.googleapis.com/ajax/libs/jquery/1.7.2/jquery.min.js. If you're in a closed network this might be the problem. Jquery not loading should show on in the dev-tools console |
Thanks for the reply. I do indeed see jquery and jquery-ui pulling in The error seems to indicate that either it doesn't know what $.JQPlugin is, On Mon, Feb 9, 2015 at 5:52 PM, CyberRodent notifications@github.com
|
Progress! D'oh! I didn't realize I was working with an older checkout. After updating my checkout I am also seeing this error. I beieved it was introduced in some recent-ish changes to the js libs. In particular it seems we now need to pull in jquery.plugin.min.js which was added in and then removed in Adding that line back in removes the error you are reporting, and I'd be happy to merge in a pull req for that. I'd also like to take a closer look at that set patches to see if there is anything else that might needs a tweak. Looks like there are some new options for locale which may require someone to also add a locale-specific file if they are using a new locale. |
as of a21b943 this should be resolved |
I see these errors in the JavaScript console when I load a morgue screen:
Uncaught TypeError: Cannot read property 'createPlugin' of undefined jquery.timeentry.min.js:6
Uncaught TypeError: undefined is not a function edit.js:27
This is on Chrome.
Morgue running behind apache, per the README
The text was updated successfully, but these errors were encountered: