refactor: Simplify bottom-left sidebar section#60370
Merged
Merged
Conversation
The health tab doesn't need to exist as a separate thing, move it inside "Help" and surface the same errors via the badges like we had before. Also, remove duplicated "Settings" that was present both in the outside and inside "Help"
rafaeelaudibert
requested review from
a team,
MattBro and
fercgomes
and removed request for
a team
This was referenced May 27, 2026
Member
Author
This was referenced May 27, 2026
![stamphog[bot]](https://avatars.githubusercontent.com/in/3000332?s=60&v=4){kind=small}
There was a problem hiding this comment.
This PR exceeded the automated size gate ceiling and has zero reviews. It's a frontend-only refactor merging HealthMenu into HelpMenu, but at 651 lines across 20 files with no reviewer sign-off, it needs at least one human to verify the UI behavior is preserved before auto-approval.
Contributor
Prompt To Fix All With AIFix the following 1 code review issue. Work through them one at a time, proposing concise fixes.
---
### Issue 1 of 1
frontend/src/lib/components/HelpMenu/HelpMenu.tsx:63-64
`totalIssues` is accessed directly from `healthSummaryLogic` here, but `helpMenuLogic` already connects to `healthSummaryLogic` and exposes `totalIssues` as a connected value. The extra `useValues(healthSummaryLogic)` import in the component is redundant — `totalIssues` can be destructured from `helpMenuLogic` alongside the other badge values it already provides.
```suggestion
const { postHogStatusTooltip, postHogStatusBadgeStatus, postHogStatusBadgeContent } = useValues(posthogStatusLogic)
const { totalIssues, isUnifiedHealthEnabled, triggerBadgeContent, triggerBadgeStatus } = useValues(helpMenuLogic)
```
Reviews (1): Last reviewed commit: "refactor: Simplify bottom-left sidebar s..." | Re-trigger Greptile |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
Contributor
|
Size Change: -3.26 kB (0%) Total Size: 80.2 MB 📦 View Changed
ℹ️ View Unchanged
|
Contributor
|
✅ Visual changes approved by @rafaeelaudibert — baseline updated in 20 changed. |
rafaeelaudibert
force-pushed
the
rafa/06-health-alerts-rollout
branch
2 times, most recently
from
e4adb6d to
e0e86f4
Compare
rafaeelaudibert
force-pushed
the
rafa/07-simplify-bottom-left-panels
branch
2 times, most recently
from
f55953d to
e02218e
Compare
Contributor
|
🎭 Playwright didn't run on this PR — your changes touch code that could affect E2E behavior, but Playwright is opt-in via label now to keep CI cost down. Add the Most PRs don't need this. Real regressions still get caught on master and fix-forward. |
rafaeelaudibert
force-pushed
the
rafa/07-simplify-bottom-left-panels
branch
from
e02218e to
f55953d
Compare
rafaeelaudibert
changed the base branch from
rafa/06-health-alerts-rollout
to
graphite-base/60370
rafaeelaudibert
force-pushed
the
graphite-base/60370
branch
from
e0e86f4 to
ff7384d
Compare
rafaeelaudibert
force-pushed
the
rafa/07-simplify-bottom-left-panels
branch
from
f55953d to
155e88d
Compare
rafaeelaudibert
force-pushed
the
rafa/07-simplify-bottom-left-panels
branch
from
155e88d to
1681eb4
Compare
20 updated Run: 4da673f3-a889-477e-85c3-8e981f4ea01a Co-authored-by: rafaeelaudibert <32079912+rafaeelaudibert@users.noreply.github.com>
Contributor
|
🎭 Playwright report · View test results →
These issues are not necessarily caused by your changes. |
rafaeelaudibert
force-pushed
the
rafa/07-simplify-bottom-left-panels
branch
from
484a4c7 to
1416607
Compare
fercgomes
approved these changes
May 28, 2026
Member
Author
Merge activity
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The
HealthMenucomponent and its associated logic have been consolidated into theHelpMenu. Rather than maintaining a separate menu entry in the nav bar footer, health status information (PostHog platform status, health issue counts, and badge indicators) is now surfaced directly within the Help menu.Key changes:
HealthMenucomponent and itshealthMenuLogic, replacing them withposthogStatusLogic(extracted status selectors) andhealthSummaryLogic(renamed fromunifiedHealthMenuLogic, with badge selectors moved up tohelpMenuLogic)g hkeyboard shortcut for toggling the health menuincidentStatusLogictypes to remove theIncidentIoprefix, and renamed mock fixture files and theINCIDENT_IO_STATUS_PAGE_BASEconstant to remove vendor-specific namingPosthogStatusShownOnlyIfNotOperationalcomponent now sources its values fromposthogStatusLogicinstead ofhealthMenuLogic