-
Notifications
You must be signed in to change notification settings - Fork 39
Feat/sso via portal #542
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Feat/sso via portal #542
Conversation
Warning Rate limit exceeded@clairekinde11 has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 23 minutes and 19 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (1)
WalkthroughAdds three new documentation pages for self-serve SSO setup and management and updates the sidebar to include a “Self-serve SSO” section. Content covers enabling per-organization access, adding/configuring SSO via a self-serve portal, and managing/troubleshooting connections. No code logic changes. Changes
Sequence Diagram(s)sequenceDiagram
autonumber
actor CustomerAdmin as Customer Admin
participant App as Self-serve Portal (App)
participant IdP as Identity Provider
participant Kinde as Kinde
CustomerAdmin->>App: Open self-serve portal (SSO section)
App->>CustomerAdmin: Prompt for verified domain + IdP config
CustomerAdmin->>IdP: Create app, get metadata (issuer, cert, etc.)
CustomerAdmin->>App: Enter domain + IdP details
App-->>Kinde: Create enterprise connection (pending)
Kinde-->>App: Connection created (requires enablement)
CustomerAdmin->>Kinde: Admin reviews and finishes setup
Kinde-->>CustomerAdmin: Connection ready
CustomerAdmin->>Kinde: Enable connection for org/users
Note over Kinde,App: Users can authenticate via configured IdP
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
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 |
Deploying kinde-docs-previews with
|
Latest commit: |
bd907f3
|
Status: | ✅ Deploy successful! |
Preview URL: | https://dfac230b.kinde-docs-previews.pages.dev |
Branch Preview URL: | https://feat-sso-via-portal.kinde-docs-previews.pages.dev |
I got the attached screen when clicking on either of Cloudlfare links
Maybe it's showing this because it's in draft - if that's the case then we can ignore this. |
Deploying kinde-docs-preview with
|
Latest commit: |
775de82
|
Status: | ✅ Deploy successful! |
Preview URL: | https://a0ebf26f.kinde-docs-preview.pages.dev |
Branch Preview URL: | https://feat-sso-via-portal.kinde-docs-preview.pages.dev |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 2
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
src/content/docs/authenticate/self-serve-sso/add-sso-self-serve.mdx
(1 hunks)src/content/docs/authenticate/self-serve-sso/manage-self-serve-connections.mdx
(1 hunks)src/content/docs/authenticate/self-serve-sso/self-manage-sso-per-org.mdx
(1 hunks)src/data/sidebarData.ts
(1 hunks)
🔇 Additional comments (3)
src/data/sidebarData.ts (1)
154-158
: Self-serve SSO nav placement looks spot on.The label matches what folks will search for and the directory slug lines up cleanly with the new doc set. Nice positioning between Enterprise connections and MFA.
src/content/docs/authenticate/self-serve-sso/self-manage-sso-per-org.mdx (1)
32-36
: Upgrade gating callout is clear.Thanks for flagging Scale-plan availability up front; that saves readers guesswork before diving into the flow.
src/content/docs/authenticate/self-serve-sso/manage-self-serve-connections.mdx (1)
41-49
: Solid troubleshooting checklist.This covers the usual SAML gotchas (ACS URL, domain, certs, upstream params) and gives support plenty to run through before escalating—nicely done.
src/content/docs/authenticate/self-serve-sso/add-sso-self-serve.mdx
Outdated
Show resolved
Hide resolved
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Draft PR for docs review by Dave.
Summary by CodeRabbit