Skip to content

feat(explore): show start + finish timestamp in more legible utc - #123238

Open
adrianviquez wants to merge 1 commit into
masterfrom
av/show-explore-timestamps-in-utc
Open

feat(explore): show start + finish timestamp in more legible utc#123238
adrianviquez wants to merge 1 commit into
masterfrom
av/show-explore-timestamps-in-utc

Conversation

@adrianviquez

Copy link
Copy Markdown
Contributor

Making trace attributes like precise.start_ts and precise.finish_ts show up as readable UTC dates instead of raw Unix timestamps for readability, as per #111412. Added tests to confirm the timestamps render correctly and gracefully fall back to the raw value if it isn't a valid number.

After
Screenshot 2026-08-31 at 1 40 56 PM

Closes EXP-848

@adrianviquez
adrianviquez requested review from a team as code owners August 31, 2026 20:41
@linear-code

linear-code Bot commented Aug 31, 2026

Copy link
Copy Markdown

EXP-848

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Aug 31, 2026
if (!Number.isFinite(seconds)) {
return fallback;
}
return <DateTime utc year seconds milliseconds date={seconds * 1000} />;

@ryan953 ryan953 Aug 31, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

might also be helpful to users to wrap this with a tooltip & https://sentry.sentry.io/scraps/core/relativetime/ inside it

import type {TraceTree} from 'sentry/views/performance/newTraceDetails/traceModels/traceTree';
import {makeTracesPathname} from 'sentry/views/traces/pathnames';

export function renderPreciseTimestamp(

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can this go into static/app/utils/date or something similar? doesn't seem like it's doing anything that's perf or trace or eap specific. someone could find and re-use it in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants