Skip to content

Hide count badge on Runs inbox tab#2748

Merged
Twixes merged 3 commits into
mainfrom
posthog-code/hide-runs-tab-count
Jun 18, 2026
Merged

Hide count badge on Runs inbox tab#2748
Twixes merged 3 commits into
mainfrom
posthog-code/hide-runs-tab-count

Conversation

@andrewm4894

Copy link
Copy Markdown
Member

Problem

The count badges on the Inbox tabs put numbers in people's faces. The PR and Reports counts are meaningful, but the Runs count is just noise.

Changes

Stop rendering the count badge on the Runs tab. Pull requests and Reports tabs keep their counts.

How did you test this?

Not run — single-line conditional change to badge rendering in InboxTabBar.

Automatic notifications

  • Publish to changelog?
  • Alert Sales and Marketing teams?

Created with PostHog Code from a Slack thread

The run count was noise rather than signal. Keep counts on Pull requests and Reports tabs, but stop showing the number on the Runs tab.

Generated-By: PostHog Code
Task-Id: d6796ead-a2fb-4d8a-a63f-3586a50c750f
@github-actions

github-actions Bot commented Jun 18, 2026

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit b406efc.

@greptile-apps

greptile-apps Bot commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Reviews (1): Last reviewed commit: "Hide count badge on Runs inbox tab" | Re-trigger Greptile

@andrewm4894 andrewm4894 marked this pull request as ready for review June 18, 2026 09:27
@andrewm4894 andrewm4894 self-assigned this Jun 18, 2026
@andrewm4894 andrewm4894 added the Stamphog This will request an autostamp by stamphog on small changes label Jun 18, 2026
github-actions[bot]
github-actions Bot previously approved these changes Jun 18, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Trivial cosmetic change — adds a tab key guard to suppress a count badge on the "runs" inbox tab. No logic, data model, or API contract changes.

The Runs tab no longer shows a count and nothing else consumed it, so remove `runs` from `InboxTabCounts` rather than carry an unused field. Also drops the `runs_count` analytics property that was only fed from it.

Generated-By: PostHog Code
Task-Id: d6796ead-a2fb-4d8a-a63f-3586a50c750f
@github-actions github-actions Bot dismissed their stale review June 18, 2026 09:32

New commits pushed (delta classified non_trivial_delta) — stamphog approval dismissed; re-review running automatically.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Coherent, low-risk cosmetic product change: suppresses the count badge on the Runs inbox tab and removes the now-unused runs field from the tab-count type, analytics schema, and computation logic. All test updates are consistent with the removal, and the optional analytics field deletion is non-breaking.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7c04d2d6c4

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread packages/core/src/inbox/engagement.ts
The leftover engagement test fixtures still passed `runs` in `tabCounts`, which are excess-property errors against the narrowed type and failed `@posthog/core` typecheck in CI before tests could run.

Generated-By: PostHog Code
Task-Id: d6796ead-a2fb-4d8a-a63f-3586a50c750f
@stamphog

stamphog Bot commented Jun 18, 2026

Copy link
Copy Markdown

Retaining stamphog approval — delta since last review classified as trivial_paths.

@Twixes Twixes enabled auto-merge (squash) June 18, 2026 10:10
@Twixes Twixes merged commit e1ba833 into main Jun 18, 2026
23 checks passed
@Twixes Twixes deleted the posthog-code/hide-runs-tab-count branch June 18, 2026 10:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Stamphog This will request an autostamp by stamphog on small changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants