Elgato_dark/hyperdx
1
0
Fork 0
mirror of https://github.com/hyperdxio/hyperdx synced 2026-04-21 13:37:15 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 6

fix: show alert indicator for bar charts too (#1487)

Browse source 
Fixes: HDX-2722
This commit is contained in:
Tom Alexander 2025-12-16 13:40:12 -05:00 committed by GitHub
parent 80117ebf9a
commit dc846011f3
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
3 changed files with 11 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
.changeset/odd-llamas-shave.md Normal file
View file

@ -0,0 +1,5 @@
---
"@hyperdx/app": patch
---
fix: show alert indicator for bar charts too

5
packages/api/src/tasks/checkAlerts/index.ts
View file

@ -289,7 +289,10 @@ const getChartConfigFromAlert = (
} else if (details.taskType === AlertTaskType.TILE) {
const tile = details.tile;
// Doesn't work for metric alerts yet
if (tile.config.displayType === DisplayType.Line) {
if (
tile.config.displayType === DisplayType.Line ||
tile.config.displayType === DisplayType.StackedBar
) {
return {
connection,
dateRange,

3
packages/app/src/DBDashboardPage.tsx
View file

@ -258,7 +258,8 @@ const Tile = forwardRef(
</Text>
{hovered ? (
<Flex gap="0px">
{chart.config.displayType === DisplayType.Line && (
{(chart.config.displayType === DisplayType.Line ||
chart.config.displayType === DisplayType.StackedBar) && (
<Indicator
size={alert?.state === AlertState.OK ? 6 : 8}
zIndex={1}