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

Analytics error events #577

Merged
merged 4 commits into from
Feb 2, 2015
Merged

Analytics error events #577

merged 4 commits into from
Feb 2, 2015

Conversation

Christof
Copy link
Contributor

@Christof Christof commented Jan 6, 2015

Create google analytics events for all exceptions. The category is always error, the action is the error message including the cause and the label is the stack trace.
Normal error handling is resumed after the event has been posted.

@Christof
Copy link
Contributor Author

Christof commented Jan 6, 2015

@LFDM Is it possible to create exceptions outside the scope of angular? If it is possible, I would like to have an example to test the window.onerror handler. Is there another place to create the GlobalErrorHandler? I think the ArethusaCtrl might be too late in the application life cycle.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.0%) when pulling f970e4e on analytics-error-events into a8abcd3 on deploy_jan6.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) when pulling f927fa0 on analytics-error-events into a8abcd3 on deploy_jan6.

balmas added a commit that referenced this pull request Feb 2, 2015
…vents

Analytics error events

I'm merging this because I need it to help me debug a problem Vanessa is having.
@balmas balmas merged commit d6e3bf6 into deploy_jan6 Feb 2, 2015
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

Successfully merging this pull request may close these issues.

None yet

3 participants