Skip to content

add gitlab as an sso provider#7

Closed
masnwilliams wants to merge 2 commits into
mainfrom
hypeship/add-gitlab-sso
Closed

add gitlab as an sso provider#7
masnwilliams wants to merge 2 commits into
mainfrom
hypeship/add-gitlab-sso

Conversation

@masnwilliams
Copy link
Copy Markdown
Contributor

@masnwilliams masnwilliams commented May 12, 2026

Summary

Adds GitLab to the list of recognized SSO providers in getSSOProviderInfo, so any provider key containing gitlab renders with the GitLab label and tanuki mark.

  • New GitLabMark SVG icon in icons.tsx.
  • Mapping entry for gitlab in sso-provider.tsx.

Test plan

  • bun run typecheck
  • bun run --filter '@onkernel/managed-auth-react' build
  • Verify in demo that a gitlab provider key renders the correct label and icon

Note

Low Risk
Low risk UI-only change: adds a new SVG icon and a simple gitlab string match in getSSOProviderInfo, with no auth flow or data-handling logic modified.

Overview
Adds GitLab as a recognized SSO provider in managed-auth-react so provider keys containing gitlab render with a GitLab label and the new GitLabMark icon.

Introduces the GitLabMark SVG in icons.tsx and wires it into getSSOProviderInfo in sso-provider.tsx.

Reviewed by Cursor Bugbot for commit b97599b. Bugbot is set up for automated code reviews on this repo. Configure here.

@masnwilliams masnwilliams marked this pull request as ready for review May 12, 2026 20:16
@firetiger-agent
Copy link
Copy Markdown

Firetiger deploy monitoring skipped

This PR didn't match the auto-monitor filter configured on your GitHub connection:

Any PR that changes the kernel API. Monitor changes to API endpoints (packages/api/cmd/api/) and Temporal workflows (packages/api/lib/temporal) in the kernel repo

Reason: PR modifies SSO provider UI components in managed-auth-react, not kernel API endpoints or Temporal workflows.

To monitor this PR anyway, reply with @firetiger monitor this.

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