Sentry destination update #4415
Conversation
|
|
||
| Since Sentry is an error tracking application, we need to make sure to send something broken from the website to see the event appear in Sentry dashboard. For example, firing the following error event from the web console, Sentry.captureException(new Error("Some error"));we can see the relevant event payload with the event_id information as below - | ||
|
|
||
|  |
There was a problem hiding this comment.
@vanand17 is it necessary to include this image? The image quality is also very poor
There was a problem hiding this comment.
@stayseesong I think the image is ok to include here, since it's not specific to any UI, but rather shows where to find an event_id in the chrome inspector.
@vanand17 Can you supply a full resolution image that is not marked up?
|
|
||
|  | ||
|
|
||
| This error will appear in Sentry with the same event_id we saw on the web console. This helps verify that we have the correct implementation of the destination. |
There was a problem hiding this comment.
| This error will appear in Sentry with the same event_id we saw on the web console. This helps verify that we have the correct implementation of the destination. | |
| The error appears in Sentry with the same `event_id` seen on the web console. This helps to verify that Segment has the correct implementation of the destination. |
|
@vanand17 can you send a full screen, not marked up version of the screenshot you've added to me Slack? I think GitHub may have compressed or shrunk it. |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
Co-authored-by: stayseesong <[email protected]>
|
Hi @markzegarelli and @stayseesong I did not want to add the entire page as it has my test website name on it, and we thought it would be better to avoid having it in Public documentation. Is there any other way I can provide you with the screenshot? |
|
@vanand17 can you try taking the screenshot on your computer screen rather than from your external monitor? The resolution comes out better this way |
|
@vanand17 We will crop the screenshot to a final size, we just want the original size to start with. |
|
Thanks. Here is the screenshot from my laptop screens and not my external monitor. Please let me know if I need to update it in the PR itself |
Co-authored-by: stayseesong <[email protected]>
|
Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site. |
Proposed changes
We have had multiple cases of customers questioning how Sentry destination works when added as a destination since it doesn't report errors out of the box. Since it is an error tracking tool, I feel we could have the following section in our docs for clarification.
Merge timing
NA
Related issues (optional)
NA