feat(admin): enterprise support channels and honor SSO enforce - #3229
feat(admin): enterprise support channels and honor SSO enforce#3229riderx wants to merge 6 commits into
Conversation
Optional SSO hid password login and locked teams out. Admin-set Slack/Discord/Teams links plus adoption charts stay off the member write path. Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
|
Warning Review limit reached
On-demand reviews are free for the next 23 days. After that, they cost $0.25 per reviewed file. Or wait 46 minutes for your next included review. View limit detailsLimit details: You’ve used the included review currently available. Your 65 included PR review attempts over the past 7 days set your current allowance at 1 review per hour. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (25)
Comment |
Merging this PR will degrade performance by 11.07%
|
| Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|
| ❌ | sha256 checksum for medium bundle payload |
1.3 ms | 1.5 ms | -11.07% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing fix/sso-enforce-and-enterprise-channel (2fd6c08) with main (3346033)
Footnotes
-
2 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩
Visual diff passedVisual changesGenerated at 2026-08-28T14:17:24.735Z. Threshold: 0.1% pixel difference.
Commit: Open |
Co-authored-by: Cursor <cursoragent@cursor.com>
Sonar duplication on the three ChartCard blocks blocked the quality gate. Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Sonar duplicated verifyAdmin against admin_credits. Co-authored-by: Cursor <cursoragent@cursor.com>
|
|
@coderabbitai review |
|



Summary (AI generated)
Motivation (AI generated)
Active SSO with
enforce_sso = falsestill forced SSO on login, so a broken SAML IdP could lock a team out. Enterprise ops also needed a dedicated support channel per org and a way to see SSO and channel adoption over time.Business Impact (AI generated)
Stops avoidable enterprise lockouts when SSO is configured but not required. Makes it obvious which paid Enterprise orgs have SSO and a dedicated support channel, so Capgo can drive adoption of those features.
Test Plan (AI generated)
tests/admin-org-support-channel.test.ts: 401, not_admin, admin set/clear, client write blocked, adoption trendtests/admin-stats.unit.test.tsacceptsenterprise_adoptionhas_ssoandenforce_sso: falsecan still sign in with passwordScreenshots (AI generated)
Optional SSO still allows password login:
Enforced SSO hides password login:
Admin enterprise charts (SSO, support channel, adoption):
Org settings show the Capgo-managed support channel:
Generated with AI
Made with Cursor
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.