-
Notifications
You must be signed in to change notification settings - Fork 735
IE 8: Version mismatch; please ensure you're using the latest version of the Mixpanel code snippet.
#81
Comments
+1 |
Gah super weird. Looking into this now! Sorry for the trouble |
What version of analytics.js are you using btw? |
0.2.2 |
Ah, can you add the latest version and see if it's still there? We upgraded the Mixpanel snippet in 0.3.4 to use their version 2.2, so that could easily be it. |
Tried with 0.5.0, same error. |
Any chance that you guys have some code that can reproduce this reliably? I've seen the error you're talking about on a few occasions, so there is definitely a bug somewhere. I'm finding it to be pretty intermittent which is making it difficult to test in IE8. My best guess is that it has something to do with how the mixpanel object gets loaded on IE8. I'm curious whether moving the If you have a gist or a page I can look at which replicates the problem consistently, that would be incredibly helpful. |
I first noticed the warning when trying to debug #80 and since we implemented the workaround for that issue I haven't been able to reproduce this. |
@paulyoung - cool! I'm not sure how the two problems would be related at the moment, but I'm glad to hear that it seems to have fixed things. :) @pheuter - do you happen to have a consistent test case which fails independent of KM? If there is still a problem, I'd love to fix it asap. Thanks for your help! |
@calvinfo Unfortunately I do not. Like @paulyoung mentioned, it has been difficult to reproduce since #80 was fixed, though it seems unlikely the two had anything in common... |
Hey guys, if you can reproduce let us know and we'll check it out more. Gonna go ahead and close for now |
I have reproduced the issue at http://www.reverbnation.com in IE8. I was able to debug it in a failed state and found IE was loading the mixpanel-2.2.min.js as soon as the statement d.parentNode.insertBefore(b,d); was run before the rest of the snippet had finished. Thus it was loading and running the script before it had set the version with the statement a.__SV=1.2;. If I move the statement d.parentNode.insertBefore(b,d); to the end of the snippet the problem goes away. Apparently IE8 can load the script as soon as it is inserted in the DOM. |
@rshubbard what version of analytics.js are you using? I had guessed this might be an issue. We updated the snippet in v0.9.5 to set |
Sorry my issue was with Mixpanel itself. I have forwarded them the issue. Appears to only happen on IE8. |
We have started getting this error again on IE8 today, did Mixpanel change something just recently? Because I think it was fine until last week and today we just saw this come up in our IE8 console. Mixpanel error: |
When attempting to use Analytics.js with Mixpanel on IE 8, the following error is thrown:
Version mismatch; please ensure you're using the latest version of the Mixpanel code snippet.
This does not occur on any other browser...
The text was updated successfully, but these errors were encountered: