posthog icon indicating copy to clipboard operation
posthog copied to clipboard

fix(alerts): show the interval alert will check

Open anirudhpillai opened this issue 7 months ago â€ĸ 2 comments

[!IMPORTANT] 👉 Stay up-to-date with PostHog coding conventions for a smoother review.

Problem

Users find it confusing that setting alert to run 'hourly' doesn't check the last hour on the trend insight

Changes

Explicitly show the interval alert will check. image image

Did you write or update any docs for this change?

How did you test this code?

Tested locally

anirudhpillai avatar Jun 12 '25 13:06 anirudhpillai

📸 UI snapshots have been updated

3 snapshot changes in total. 0 added, 3 modified, 0 deleted:

Triggered by this commit.

👉 Review this PR's diff of snapshots.

posthog-bot avatar Jun 12 '25 13:06 posthog-bot

Size Change: 0 B

Total Size: 2.57 MB

â„šī¸ View Unchanged
Filename Size
frontend/dist/toolbar.js 2.57 MB

compressed-size-action

github-actions[bot] avatar Jun 12 '25 13:06 github-actions[bot]

📸 UI snapshots have been updated

2 snapshot changes in total. 0 added, 2 modified, 0 deleted:

  • chromium: 0 added, 2 modified, 0 deleted (diff for shard 2)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

posthog-bot avatar Jun 17 '25 07:06 posthog-bot

📸 UI snapshots have been updated

1 snapshot changes in total. 0 added, 1 modified, 0 deleted:

  • chromium: 0 added, 1 modified, 0 deleted (diff for shard 7)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

posthog-bot avatar Jun 17 '25 09:06 posthog-bot

📸 UI snapshots have been updated

1 snapshot changes in total. 0 added, 1 modified, 0 deleted:

[!CAUTION]

Detected flapping snapshots

These snapshots have auto-updated more than once since the last human commit:

  • scenes-app-surveys--survey-templates--dark.png (chromium, shard 7)

The flippy-flappies are deadly and must be fixed ASAP. They're productivity killers. Run pnpm storybook locally and make the fix now. (Often, the cause is ResizeObserver being used instead of the better CSS container queries.)

  • chromium: 0 added, 1 modified, 0 deleted (diff for shard 7)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

posthog-bot avatar Jun 17 '25 09:06 posthog-bot