Repository navigation
Make GitHub integration free for all users - #3152
Merged
Merged
Conversation
Contributor
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (6)
📝 WalkthroughWalkthroughRemoved GitHub feature-flag gating and made GitHub integration visible on the free plan; updated billing comparison rows; replaced GitHub demo assets with Slack in the paywall preview and adjusted synced item type/display. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Contributor
Greptile SummaryThis PR makes the GitHub integration freely available to all users by removing the PostHog Key changes:
Confidence Score: 5/5
Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[User opens Integrations] --> B{Platform?}
B -->|Web| C[integrations/page.tsx]
B -->|Desktop| D[IntegrationsSettings.tsx]
C --> E{Integration?}
E -->|GitHub| F[Always visible ✅]
E -->|Slack| G{hasSlackAccess flag?}
G -->|Yes| H[Show Slack card]
G -->|No| I[Hidden]
E -->|Linear| J[Show Linear card via isItemVisible]
D --> K{Integration?}
K -->|GitHub| L[isItemVisible only — no flag gate ✅]
L --> M[onManage → open web directly ✅]
K -->|Slack| N{hasSlackAccess flag?}
N -->|Yes| O[Show → gateFeature paywall]
K -->|Linear| P[isItemVisible → gateFeature paywall]
subgraph Paywall Preview Updated
Q[IntegrationsDemo] --> R[Linear ↔ Slack ✅]
end
subgraph Billing Updated
S[Free plan features] --> T[+ GitHub integration ✅]
U[Comparison table] --> V[GitHub = Free ✅]
U --> W[Slack = Pro/Enterprise ✅]
end
Reviews (1): Last reviewed commit: "feat: make GitHub integration free for a..." | Re-trigger Greptile |
Remove the PostHog feature flag gate and desktop paywall from the GitHub integration so it is accessible to all plan tiers. Update billing plans to reflect GitHub as a free feature and replace GitHub with Slack in the pro features paywall preview.
AviPeltz
force-pushed
the
github-app-integration-paywall-access
branch
from
April 3, 2026 18:52
a80144e to
f99e86f
Compare
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
Contributor
🧹 Preview Cleanup CompleteThe following preview resources have been cleaned up:
Thank you for your contribution! 🎉 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
🧹 Nitpick comments (1)
apps/desktop/src/renderer/routes/_authenticated/settings/billing/plans/page.tsx (1)
138-157: Consider a single source for integration entitlements.
PLANS.free.featuresnow carries GitHub, while this comparison matrix separately hard-codes GitHub and Slack availability. Since other billing views already renderplan.features, keeping both structures aligned will get brittle quickly. Consider deriving these rows from shared plan-entitlement data instead of duplicating them here.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@apps/desktop/src/renderer/routes/_authenticated/settings/billing/plans/page.tsx` around lines 138 - 157, The comparison matrix duplicates integration entitlements that already live on PLANS (e.g., PLANS.free.features) and plan.features; remove the hard-coded rows ("GitHub integration", "Slack integration", etc.) and instead derive those rows from a single source of truth by mapping the shared plan entitlement data (PLANS[*].features or plan.features) into the comparison rows; implement a small mapper or helper used by the page component to produce label, values and comingSoon from the feature keys so the matrix always reflects PLANS entries and avoids divergence.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In
`@apps/desktop/src/renderer/routes/_authenticated/settings/billing/plans/page.tsx`:
- Around line 138-157: The comparison matrix duplicates integration entitlements
that already live on PLANS (e.g., PLANS.free.features) and plan.features; remove
the hard-coded rows ("GitHub integration", "Slack integration", etc.) and
instead derive those rows from a single source of truth by mapping the shared
plan entitlement data (PLANS[*].features or plan.features) into the comparison
rows; implement a small mapper or helper used by the page component to produce
label, values and comingSoon from the feature keys so the matrix always reflects
PLANS entries and avoids divergence.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: ff75364d-27b7-4dec-95d8-aca5ecefd3aa
📒 Files selected for processing (6)
apps/desktop/src/renderer/components/Paywall/components/FeaturePreview/components/IntegrationsDemo/IntegrationsDemo.tsxapps/desktop/src/renderer/components/Paywall/constants.tsapps/desktop/src/renderer/routes/_authenticated/settings/billing/constants.tsapps/desktop/src/renderer/routes/_authenticated/settings/billing/plans/page.tsxapps/desktop/src/renderer/routes/_authenticated/settings/integrations/components/IntegrationsSettings/IntegrationsSettings.tsxapps/web/src/app/(dashboard-legacy)/integrations/page.tsx
MocA-Love
pushed a commit
to MocA-Love/superset
that referenced
this pull request
Apr 5, 2026
Remove the PostHog feature flag gate and desktop paywall from the GitHub integration so it is accessible to all plan tiers. Update billing plans to reflect GitHub as a free feature and replace GitHub with Slack in the pro features paywall preview.
5 tasks
MocA-Love
added a commit
to MocA-Love/superset
that referenced
this pull request
Apr 5, 2026
cherry-pick方式で内容を取り込み済みの14コミットをgit履歴上もマージ済みにする。 取り込み済み (cherry-pick / 手動移植): - be22b46 ParadevAI#3125 — スキップ (下記参照) - 88bc7fb ParadevAI#3127 — Revert DA1 ✓ - 92d0ff9 ParadevAI#3054 — DA1 fix ✓ - c48450e ParadevAI#3093 — file viewer pane fix ✓ - fffa8db ParadevAI#3128 — version 1.4.7 ✓ - 589a7c7 ParadevAI#3136 — fuzzy scorer (ハイブリッド方式) ✓ - ceb8c81 ParadevAI#3150 — Electron 40.8.5 ✓ - 8922b94 ParadevAI#3137 — terminalId分離 ✓ - c7508e5 ParadevAI#3152 — GitHub無料化 ✓ - 2b91f11 ParadevAI#3155 — v2 terminal theme ✓ - b8b11af ParadevAI#3154 — TUI dimension fix ✓ - 7599ace ParadevAI#3149 — v2 sidebar file tree (手動統合) ✓ - 4d7c612 ParadevAI#3174 — DnD重複削除 ✓ - 864977d ParadevAI#3157 — Host Service分離 ✓ 意図的にスキップ: - be22b46 ParadevAI#3125 (GitHub polling簡素化) フォーク独自のGitHubSyncService (バックエンド集中ポーリング) と 設計が異なるため不採用。upstreamはフロントエンドhover駆動、フォークは バックエンドキャッシュウォーマー方式。詳細は githubQueryPolicy.ts と github-sync-service.ts のFORK NOTEを参照。 ゴースト・マージ復元 (revert 134cfd5 で消失した内容): - 538f306 ParadevAI#3120 — Patch vuln ✓ - 1588d20 ParadevAI#3108 — terminal lifecycle分離 ✓ - 59426f6 ParadevAI#3122 — file tree + FilePane + Alert refactor (手動統合) ✓ - 10d9a5d ParadevAI#3097 — tiptap line-height ✓ - 337a9ae ParadevAI#3121 — Codex hooks削除 ✓
5 tasks
saddlepaddle
added a commit
that referenced
this pull request
Aug 28, 2026
The PostHog flag has been at 100% with no readers since the GitHub integration went free for all (#3152); it is archived alongside slack-integration-access.
saddlepaddle
added a commit
that referenced
this pull request
Aug 28, 2026
…triggers are (#6933) * feat(integrations): offer trigger-only integrations only where their triggers are Notion, Microsoft Teams, Sentry and Google exist to feed automation triggers, which the automation-event-triggers flag gates to @superset.sh, yet every user saw their Connect buttons on desktop settings and web /integrations. Visibility now derives from the same flag payload: GitHub, Slack and Linear are standalone (repos/PRs, the Slack app, tasks) and always offered; the rest are offered while one of their trigger kinds is in the payload, and their web pages 404 otherwise. * chore(shared): drop the unused GITHUB_INTEGRATION_ACCESS flag constant The PostHog flag has been at 100% with no readers since the GitHub integration went free for all (#3152); it is archived alongside slack-integration-access.
This branch was previously deployed
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
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.
Summary
GITHUB_INTEGRATION_ACCESS) from the GitHub integration on both web and desktop, making it visible to all usersgateFeature) from the GitHub integration's "Manage" button so free users can access it directlyTest plan
Summary by cubic
Make the GitHub integration free for all users by removing the feature flag and desktop paywall. Update plans and the paywall preview to show Slack as the gated integration instead.
GITHUB_INTEGRATION_ACCESSon web and desktop; GitHub card is always visible.gateFeatureon GitHub Manage; free users can connect/manage directly.Written for commit f99e86f. Summary will update on new commits.
Summary by CodeRabbit
New Features
Bug Fixes