Skip to content

docs: clarify connector vs. SSO sign-in - #29

Merged
mindymo merged 1 commit into
mainfrom
docs/sso-connector-disambiguation-note
Aug 31, 2026
Merged

docs: clarify connector vs. SSO sign-in#29
mindymo merged 1 commit into
mainfrom
docs/sso-connector-disambiguation-note

Conversation

@mindymo

@mindymo mindymo commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds a note to the top of the connector doc distinguishing this connector (identity governance sync/provisioning for C1) from signing in to C1 via this provider's SSO. Customers have been conflating the two.

Context

The connector doc and the C1 tenant sign-in flow both involve the same identity provider, but they're unrelated setups (separate app registrations/OAuth clients in most cases). This note links to the specific SSO setup section for this provider so readers land in the right place.

Add a note distinguishing this connector (identity governance sync/provisioning) from signing in to C1 via this provider's SSO, linking to the tenant setup guide. Customers have been conflating the two.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

Connector PR Review: docs: clarify connector vs. SSO sign-in

Blocking Issues: 0 | Suggestions: 0 | Threads Resolved: 0
Criteria: Criteria status: loaded .claude/skills/ci-review.md from trusted base 526788bfb7cb.
Review mode: full
View review run

Review Summary

The full PR diff was scanned for security and correctness. The change is documentation-only: a four-line Note block added to docs/connector.mdx distinguishing the OneLogin connector from OneLogin SSO sign-in to C1, with a relative link to the sign-in setup section. No Go source, config, provisioning, or dependency manifest (go.mod/go.sum) files were touched, so no connector implementation, breaking-change, or dependency criteria apply; the relative-link style matches the existing /product/admin/applications link already in this file. No prior findings existed and no new issues were found.

Security Issues

None found.

Correctness Issues

None found.

Suggestions

None.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No blocking issues found.

@mindymo
mindymo merged commit 71008a1 into main Aug 31, 2026
10 of 11 checks passed
@mindymo
mindymo deleted the docs/sso-connector-disambiguation-note branch August 31, 2026 23:25
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