Skip to content

Add sentry JS

Luke Bennett requested to merge add-sentry-js-again-with-vue into master

What does this MR do?

Adds sentry JS

EE port: https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/1667

Are there points in the code the reviewer needs to double check?

Why was this MR needed?

We need to be able to monitor the health of our JS.

Screenshots (if relevant)

Screen_Shot_2016-10-09_at_23.48.13

The top error is an ajax error (I know we already monitor the backend using sentry so any ajax errors should already be picked up, but I don't think that makes this useless).

The second error is a manual test.

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

/cc @jschatz1

Merge request reports