Skip to content

fix(github): strip platform-setup CTA from user settings page - #9

Merged
ralyodio merged 1 commit into
masterfrom
fix/github-settings-platform-strip
May 22, 2026
Merged

ralyodio merged 1 commit into
masterfrom
fix/github-settings-platform-strip

Conversation

@ralyodio

Copy link
Copy Markdown
Contributor

Summary

Cleans up the last piece of platform-specific UI on the per-user GitHub settings page. The "Run one-click setup" CTA was a big button that showed (admin-only) when the integration wasn't configured — but a per-user surface shouldn't have a platform-configuration button, even gated.

Replaced with a small admin-only inline link so admins can still navigate to `/admin/github/setup` from any settings page; everyone else sees a clean "integration not available yet" notice with no CTA.

Visual diff

Before (admin view, not-configured state):

  • H2 "Integration not configured"
  • Paragraph explanation
  • Big primary button "Run one-click setup (admin)"

After:

  • H2 "Integration not available yet"
  • Paragraph "check back soon"
  • Small muted line (admin only): "Admin: register the App at /admin/github/setup."

Test plan

  • Visit `/settings/integrations/github` with the GitHub App env unset → "Integration not available yet" notice. Non-admin: no CTA. Admin: small inline link to `/admin/github/setup`.
  • With env set + no installations → "No installations yet" + Install button (unchanged).
  • With env set + installations connected → installation list (unchanged).

🤖 Generated with Claude Code

The /settings/integrations/github page is for end users — install the
App on your accounts, view installations, see repos. The "Run one-click
setup" button was the only piece of platform-level admin work still
showing here.

Replaced the big CTA with a small admin-only inline link so admins can
still get to /admin/github/setup from any settings page, but no one
sees a "configure the platform" button on a per-user surface.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@ralyodio
ralyodio merged commit 9b67af8 into master May 22, 2026
@ralyodio
ralyodio deleted the fix/github-settings-platform-strip branch May 22, 2026 13:24
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