Skip to content

feat(inbox): Rename Conversations to Support and add product docs links#1802

Merged
Twixes merged 1 commit intomainfrom
posthog-code/signal-sources-rename-support-and-docs-links
Apr 22, 2026
Merged

feat(inbox): Rename Conversations to Support and add product docs links#1802
Twixes merged 1 commit intomainfrom
posthog-code/signal-sources-rename-support-and-docs-links

Conversation

@posthog
Copy link
Copy Markdown
Contributor

@posthog posthog Bot commented Apr 22, 2026

Summary

Follow-up to Annika's signal-sources onboarding feedback:

The internal conversations source key/enum is unchanged — only the user-facing label (and the toast label via SOURCE_LABELS) was renamed.

Test plan

  • Open Settings → Signal sources: verify the "Support" card replaces "Conversations" and still toggles correctly
  • Click "Learn about {product}" on each PostHog product card; confirm the docs page opens in the browser and the card's toggle does not flip
  • Confirm the same cards render correctly in the onboarding Signals step
  • Error toasts for the renamed source read "Failed to toggle PostHog Support"

Created with PostHog Code

…ks in signal source config

Renames the "Conversations" PostHog source card to "Support" to match the
PostHog Cloud UI. Adds a "Learn about {product}" external link to each
PostHog product card (Error Tracking, Support, Session Replay, LLM
Analytics) so users can discover the underlying product before setting
it up as a signal source.

Generated-By: PostHog Code
Task-Id: 8c5abb90-2665-4f5a-9138-eb3d0b3b08d4
@Twixes Twixes marked this pull request as ready for review April 22, 2026 08:27
@Twixes Twixes merged commit 78e35a2 into main Apr 22, 2026
15 checks passed
@Twixes Twixes deleted the posthog-code/signal-sources-rename-support-and-docs-links branch April 22, 2026 08:28
@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Apr 22, 2026

Prompt To Fix All With AI
This is a comment left during a code review.
Path: apps/code/src/renderer/features/inbox/components/SignalSourceToggles.tsx
Line: 216-241

Comment:
**Duplicated docs-link markup (OnceAndOnlyOnce)**

The docs-link JSX in `EvaluationsSection` is an exact copy of the one now rendered by `SignalSourceToggleCard` when `docsUrl` is provided. The URL string `"https://posthog.com/docs/llm-analytics"` is also repeated twice inside the same block (once in `href`, once in `window.open`).

Consider extracting a small shared component (e.g. `DocsLink`) and reusing it here, the same way the three PostHog cards use the new `docsUrl`/`docsLabel` props:

```tsx
function DocsLink({ url, label }: { url: string; label: string }) {
  return (
    <Text size="1" style={{ color: "var(--gray-11)" }}>
      <a
        href={url}
        target="_blank"
        rel="noopener noreferrer"
        onClick={(e) => {
          e.stopPropagation();
          e.preventDefault();
          window.open(url, "_blank", "noopener");
        }}
        style={{
          color: "var(--accent-11)",
          textDecoration: "none",
          display: "inline-flex",
          alignItems: "center",
          gap: "4px",
        }}
      >
        Learn about {label}
        <ArrowSquareOutIcon size={11} />
      </a>
    </Text>
  );
}
```

Then both `SignalSourceToggleCard` and `EvaluationsSection` can simply render `<DocsLink url="..." label="..." />`.

How can I resolve this? If you propose a fix, please make it concise.

Reviews (1): Last reviewed commit: "feat(inbox): Rename Conversations to Sup..." | Re-trigger Greptile

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.

1 participant