Skip to content

Conversation

@clr182
Copy link
Contributor

@clr182 clr182 commented Nov 27, 2025

Problem

For users who are experiencing issues that are also affecting others. Makes it more obvious when users attempt to open a ticket when a status page is open.

Changes

Show a link to the status tab when there is an ongoing operational issue

When there is degraded performance:
image

When there is a partial or major outage:
image

How did you test this code?

Ran it locally and I added a PREVIEW_MODE flag that forced the alert to display even when there was no actual incident.

Changelog: (features only) Is this feature complete?

Yes

@clr182
Copy link
Contributor Author

clr182 commented Nov 27, 2025

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

@github-actions
Copy link
Contributor

github-actions bot commented Nov 27, 2025

Size Change: 0 B

Total Size: 3.41 MB

ℹ️ View Unchanged
Filename Size
frontend/dist/toolbar.js 3.41 MB

compressed-size-action

@clr182 clr182 requested review from a team November 27, 2025 16:40
@clr182
Copy link
Contributor Author

clr182 commented Nov 27, 2025

actually it looks slightly different from the original screenshots above, looks like this now with a button that opens the status page tab rather than opening the status page itself
Screenshot 2025-11-27 at 15 52 57

<div className="flex items-start gap-2">
<IconWarning className="text-warning w-5 h-5 shrink-0 mt-0.5" />
<div className="flex-1">
<p className="font-semibold mb-1">{description}</p>
Copy link
Contributor

Choose a reason for hiding this comment

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

what will the description actually include?

Copy link
Contributor

Choose a reason for hiding this comment

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

is it always something based on a count of active incidents? like 1 ongoing incident in your example?

Copy link
Contributor Author

@clr182 clr182 Nov 27, 2025

Choose a reason for hiding this comment

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

the desc is pulled from sidePanelStatusIncidentLogic.tsx and depends on the INCIDENT_IO_STATUS_PAGE flag. when it's an incident.io status it's a count based message like the examplke

Comment on lines 70 to 78
<LemonButton
type="secondary"
size="small"
fullWidth
center
onClick={() => openSidePanel(SidePanelTab.Status)}
>
View status page
</LemonButton>
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd be tempted to delete the button (it's quite big IMO) and inline the status link with a href on the current status text

Copy link
Contributor Author

Choose a reason for hiding this comment

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

maybe this instead?
image

@clr182 clr182 enabled auto-merge (squash) November 27, 2025 19:52
Comment on lines +60 to +83
<div className="flex items-start gap-2">
<IconWarning className="text-warning w-5 h-5 shrink-0 mt-0.5" />
<div className="flex-1">
<p className="font-semibold mb-1">
<span
className="cursor-pointer text-link hover:underline"
onClick={() => openSidePanel(SidePanelTab.Status)}
>
{description}
</span>
</p>
<div className="text-sm">
<p className="mb-1">We're aware of an issue that may be affecting your PostHog experience.</p>
<p className="mb-0">
You may wish to check our{' '}
<span
className="cursor-pointer text-link hover:underline"
onClick={() => openSidePanel(SidePanelTab.Status)}
>
current status
</span>{' '}
before contacting support.
</p>
</div>
Copy link
Contributor

Choose a reason for hiding this comment

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

could also be worth exploring the LemonBanner UI component (or whatever it's called!) here

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants