-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
feat(webapp): errors page polish and GA rollout #3477
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
Changes from all commits
6f5e4fc
7b5a568
0509ca5
a41f799
0f9333f
f9ab113
e807d7f
d58f401
86b12ec
95ae892
8a40631
a3ef370
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,6 @@ | ||
| --- | ||
| area: webapp | ||
| type: feature | ||
| --- | ||
|
|
||
| Ship the Errors page to all users, with a polish + bug-fix pass: pinned "No channel" item in the Slack alert channel picker, viewer-timezone alert timestamps via Slack's `<!date^>` token, Activity sparkline peak tooltip, centered loading spinner and bug-icon empty state on the error detail page, ellipsis on the Configure alerts trigger. |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -55,6 +55,7 @@ import { | |
| statusActionToastMessage, | ||
| } from "~/components/errors/ErrorStatusMenu"; | ||
| import { useToast } from "~/components/primitives/Toast"; | ||
| import { SimpleTooltip } from "~/components/primitives/Tooltip"; | ||
| import TooltipPortal from "~/components/primitives/TooltipPortal"; | ||
| import { appliedSummary, FilterMenuProvider, TimeFilter } from "~/components/runs/v3/SharedFilters"; | ||
| import { $replica } from "~/db.server"; | ||
|
|
@@ -463,7 +464,7 @@ function FiltersBar({ | |
| LeadingIcon={BellAlertIcon} | ||
| leadingIconClassName="text-alerts" | ||
| > | ||
| Configure alerts | ||
| Configure alerts… | ||
| </LinkButton> | ||
| {list && <ListPagination list={list} />} | ||
| </div> | ||
|
|
@@ -706,9 +707,14 @@ function ErrorActivityGraph({ activity }: { activity: ErrorOccurrenceActivity }) | |
| </BarChart> | ||
| </ResponsiveContainer> | ||
| </div> | ||
| <span className="-mt-1 text-xxs tabular-nums text-text-dimmed"> | ||
| {formatNumberCompact(maxCount)} | ||
| </span> | ||
| <SimpleTooltip | ||
| button={ | ||
| <span className="-mt-1 text-xxs tabular-nums text-text-dimmed"> | ||
| {formatNumberCompact(maxCount)} | ||
| </span> | ||
| } | ||
| content="Peak occurrences in a single time bucket" | ||
| /> | ||
|
Comment on lines
+710
to
+717
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🚩 SimpleTooltip introduces a nested inside a in the table cell The new Was this helpful? React with 👍 or 👎 to provide feedback. |
||
| </div> | ||
| ); | ||
| } | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.