sentry-javascript icon indicating copy to clipboard operation
sentry-javascript copied to clipboard

Add timestamps to replays based on local time

Open kpujjigit opened this issue 1 year ago • 2 comments

Problem Statement

I've a feature request that should be fairly simple but its valuable for replay investigations and maybe relevant for other companies as well. (Let me know if this is not the right place for it. )

Problem: We have a customer complain that on on 14:17 PM on 1st of January he had a certain issues, or we see under the discover tab a specific error that happen during this timestamp (the view includes timestamp).

It would be really useful for the Replay view to include the full timestamp as well to be able to easily correlate (instead of showing the text happen "X days ago" and require us to hover over each X days ago text to see the full timestamp (in some cases we can't filter by absolute date and need to go over daily range of recordings, which makes it really difficult and require us to over each tooltip one by one and check the full timestamp)

Solution Brainstorm

During the replay itself, to know go for the specific timestamp (14:17) I need to check the tooltip on the screen for the start of the replay e.g 14:09 PM, then manually calculate the time between the error and move the time by 8 minutes to discover the relevant time of issue I want to look for.

It would be really helpful if there was a UTC timestamp view on the recoding screen on top + instead of timer for the start of the recording to have a timestamp timer.

Image

Image

Image

kpujjigit avatar Aug 21 '24 16:08 kpujjigit

we'll add the timestamp, with absolute time at the current users timezone. There are several calculators online to convert between timezone but showing UTC can be confusing to most users

bruno-garcia avatar Aug 21 '24 17:08 bruno-garcia

Relates to:

  • https://github.com/getsentry/sentry/issues/52682

bruno-garcia avatar Aug 26 '24 15:08 bruno-garcia

The title of the ticket says:

Add timestamps to replays based on local time

But that's already how replay (and all of Sentry) works.

Further in the description, you state:

It would be really helpful if there was a UTC timestamp view on the recoding screen on top + instead of timer for the start of the recording to have a timestamp timer.

So the ask is to have a UTC timestamp instead of local time. As well as to show the full timestamp, instead of the "X minutes ago" notation?

bruno-garcia avatar Aug 29 '24 17:08 bruno-garcia

Hey @bruno-garcia this is more of a UI upgrade, replacing the "x days ago" with a concrete time/date. The implementation of UTC time is not a vital priority.

Furthermore, it was requested to be able to correlate actual time of the event to the duration of the replay. So in my above screenshot of the time bar, it would be helpful if the 2:39 could also display the actual time at which the event occurred, perhaps via hover.

kpujjigit avatar Aug 29 '24 20:08 kpujjigit

Do I understand correctly that this is purely a UI issue? Meaning the SDK is sending all relevant data correctly? If so, let's transfer this to the getsentry/sentry repo :)

Lms24 avatar Sep 02 '24 09:09 Lms24

Discussion moved here: https://github.com/getsentry/sentry/issues/76933

kpujjigit avatar Sep 04 '24 17:09 kpujjigit