Skip to content

feat(go): allowlist launch channel srcs - #90

Merged
ErikBjare merged 1 commit into
masterfrom
feat/go-launch-srcs
Oct 6, 2026
Merged

ErikBjare merged 1 commit into
masterfrom
feat/go-launch-srcs

Conversation

@ErikBjare

Copy link
Copy Markdown
Member

Adds newsletter, producthunt, hackernews, reddit, x, linkedin and discord to the /go/ SRC_DEFAULT allowlist (all → subscribe), so Pro links in the v0.14.0 launch posts are attributed per channel in GA4 instead of falling into other. Same pattern as #80 (release-notes) and #84 (blog).

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@ErikBjare

Copy link
Copy Markdown
Member Author

@greptileai review

@greptile-apps

greptile-apps Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

[Medium risk] Adds launch channel tracking to a redirect router.

This PR appears safe to merge.

What we checked:

  • Launch labels stay intact: KNOWN_SRC comes directly from SRC_DEFAULT. All seven labels pass the existing cleanup unchanged, and GA4 receives the accepted src.
  • Explicit destinations still win: Each new label maps to the existing subscribe URL. An allowed to value is checked before the source default.

Summary

Adds seven launch channels to /go/: newsletter, producthunt, hackernews, reddit, x, linkedin, and discord.

  • Each channel defaults to subscribe and keeps its own label in the GA4 nudge_click event.
  • Explicit destinations and the existing redirect safeguards remain unchanged.
  • No actionable issues found.

Reviews (1) · Last reviewed commit: "feat(go): allowlist launch channel srcs ..."

@ErikBjare
ErikBjare merged commit ffcb7f4 into master Oct 6, 2026
2 checks passed
@ErikBjare
ErikBjare deleted the feat/go-launch-srcs branch October 6, 2026 20:14
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