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

Async bugs caused by missing logEventWithGroups in snippet #102

Closed
fotinakis opened this issue Jan 17, 2017 · 2 comments
Closed

Async bugs caused by missing logEventWithGroups in snippet #102

fotinakis opened this issue Jan 17, 2017 · 2 comments

Comments

@fotinakis
Copy link

It seems that logEventWithGroups is missing from your <script> snippet, both here in this repo and in the Amplitude UI where you show the installation snippet (that one is also an old 3.0.1 script version).

Not having logEventWithGroups may work sometimes, but causes intermittent and hard-to-debug async problems (especially in our single-page app) where a page may break if analytics is called before the async script has actually loaded.

It seems that the new logEventWithTimestamp function is also missing from the snippet.

@djih
Copy link
Member

djih commented Jan 18, 2017

Ah apologies for that @fotinakis , we will patch and release a new version asap

@djih
Copy link
Member

djih commented Jan 19, 2017

@fotinakis the load snippet has been updated: #103, and v3.4.0 has been retagged (no need to release a new version of the script since that hasn't changed). We will send a reminder for people to update their snippet. Thanks for reporting

@djih djih closed this as completed Jan 19, 2017
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

No branches or pull requests

2 participants