-
Notifications
You must be signed in to change notification settings - Fork 55
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
Denying cookies crashes the script #44
Comments
According to MDN:
So we should be able to skip |
I am having this issue with an anonymous browser (Windows 7 / Chrome 72.0.3626). It's rare but just occurred on our production site. |
This probably can be quick fixed if we add a try-catch to |
Though denying cookies is a strong indicator that the user does not want to be tracked; I'm thinking if we should disable the ga-lite tracking then altogether? |
I have no issue with doing that... It seems like a rare case anyway. |
If your browser has cookies disabled, using
localStorage
throws an error that crashes ga-lite.ga-lite should be able to work even without localstorage.
The text was updated successfully, but these errors were encountered: