Skip to content

Conversation

@danciaclara
Copy link
Collaborator

@danciaclara danciaclara commented Nov 24, 2025

Description

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • Feature (non-breaking change which adds functionality)
  • Improvement (change that would cause existing functionality to not work as expected)
  • Code refactoring
  • Performance improvements
  • Documentation update

Screenshots and Media (if applicable)

Test Scenarios

References


Note

Refreshes SSO/OAuth docs with new configuration images, updates plan tier labels, and minor wording fixes.

  • Documentation
    • SSO (OIDC/SAML)
      • Update images: oidc-config.pngoidc-oauth.png, saml-config.pngsaml-oauth.png in plane-one/governance/authentication/custom-sso.mdx, self-hosting/govern/oidc-sso.mdx, and self-hosting/govern/saml-sso.mdx.
      • Wording: replace references to Plane One with Plane.
      • Plan badges: change from Plane One, Plane Pro to Pro, Business on self-hosted OIDC/SAML pages.
    • GitHub OAuth
      • Add configuration screenshot under "Configure Plane" (/images/custom-sso/github-oauth.png).
    • Google OAuth
      • Add configuration screenshot under "Configure Plane" (/images/custom-sso/google-oauth.png).
    • Migration Guide
      • Update workspace plan names in license delinking step (One or ProPro or Business).

Written by Cursor Bugbot for commit c40cde4. This will update automatically on new commits. Configure here.

Summary by CodeRabbit

  • Documentation
    • Refreshed branding text referencing custom SSO (no workflow changes)
    • Renamed plan tiers to "Pro" and "Business" across relevant guides
    • Updated configuration screenshots for OIDC and SAML
    • Added visual configuration guides for GitHub and Google OAuth
    • Updated workspace migration wording to reference "Pro or Business"

✏️ Tip: You can customize this high-level summary in your review settings.

@vercel
Copy link

vercel bot commented Nov 24, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
developer-docs Error Error Dec 2, 2025 2:11pm

@coderabbitai
Copy link

coderabbitai bot commented Nov 24, 2025

Walkthrough

Documentation updates: standardized branding from "Plane One" to "Plane", renamed plan tiers to "Pro" and "Business", replaced several OAuth/SAML image references, and added image frames for GitHub and Google OAuth in self-hosting guides.

Changes

Cohort / File(s) Change Summary
Branding & Plan Tier Updates
plane-one/governance/authentication/custom-sso.mdx, self-hosting/govern/oidc-sso.mdx, self-hosting/govern/saml-sso.mdx, self-hosting/manage/migrate-plane.mdx
Replaced "Plane One" wording with "Plane"; renamed plan labels from "Plane One, Plane Pro" to "Pro, Business"; adjusted descriptive sentence to remove product-specific phrasing.
OAuth / SAML Image Reference Updates
plane-one/governance/authentication/custom-sso.mdx, self-hosting/govern/oidc-sso.mdx, self-hosting/govern/saml-sso.mdx
Updated image references: oidc-config.pngoidc-oauth.png, saml-config.pngsaml-oauth.png.
Visual Documentation Additions
self-hosting/govern/github-oauth.mdx, self-hosting/govern/google-oauth.mdx
Inserted Frame elements embedding images (/images/custom-sso/github-oauth.png, /images/custom-sso/google-oauth.png) under the "Configure Plane" sections.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Docs-only, consistent patterns across files.
  • Areas to spot-check:
    • Verify referenced image files exist at the new paths.
    • Confirm wording changes preserve intended meaning in context (single-sentence replacements).
    • Ensure no accidental markdown or frontmatter edits occurred.

Poem

🐰 I hopped through docs in twilight's glow,
Swapped "One" for "Plane" in tidy rows,
Pro and Business take their place,
OAuth screenshots fall into place,
A little rabbit's work — now watch it go! ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main changes: updating SSO configuration images and adjusting product branding/wording across documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch updates-20-11-25

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 28b03bb and c40cde4.

📒 Files selected for processing (1)
  • plane-one/governance/authentication/custom-sso.mdx (3 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
  • plane-one/governance/authentication/custom-sso.mdx
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: Cursor Bugbot

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a 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

📥 Commits

Reviewing files that changed from the base of the PR and between 8139982 and 28b03bb.

⛔ Files ignored due to path filters (6)
  • images/custom-sso/github-oauth.png is excluded by !**/*.png
  • images/custom-sso/google-oauth.png is excluded by !**/*.png
  • images/custom-sso/oidc-config.png is excluded by !**/*.png
  • images/custom-sso/oidc-oauth.png is excluded by !**/*.png
  • images/custom-sso/saml-config.png is excluded by !**/*.png
  • images/custom-sso/saml-oauth.png is excluded by !**/*.png
📒 Files selected for processing (6)
  • plane-one/governance/authentication/custom-sso.mdx (1 hunks)
  • self-hosting/govern/github-oauth.mdx (1 hunks)
  • self-hosting/govern/google-oauth.mdx (1 hunks)
  • self-hosting/govern/oidc-sso.mdx (2 hunks)
  • self-hosting/govern/saml-sso.mdx (2 hunks)
  • self-hosting/manage/migrate-plane.mdx (1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
Learnt from: Prashant-Surya
Repo: makeplane/developer-docs PR: 143
File: api-reference/byoa/build-plane-app.mdx:629-657
Timestamp: 2025-09-15T07:57:58.327Z
Learning: In documentation files for makeplane/developer-docs, the maintainers prefer to keep code examples simple with placeholder values (like hardcoded credentials) rather than using environment variables or complex setup, as it makes the documentation more readable and easier to understand for users learning the API.
🔇 Additional comments (8)
self-hosting/govern/github-oauth.mdx (1)

25-25: Verify image asset exists before deployment.

Line 25 references a new image file /images/custom-sso/github-oauth.png. This image must exist in the public/images/custom-sso/ directory for the documentation to render correctly, otherwise users will see a broken image placeholder.

The shell script above will verify whether this and related OAuth configuration images exist in the repository.

plane-one/governance/authentication/custom-sso.mdx (1)

5-5: LGTM: Branding standardization is consistent across SSO documentation.

The text change from "Plane One enables custom SSO" to "Plane enables custom SSO" aligns with the broader PR objective to standardize product branding. This is a non-functional, low-risk update that improves consistency across the documentation set.

self-hosting/manage/migrate-plane.mdx (1)

20-21: LGTM: Plan tier terminology updated accurately.

The change from "One or Pro" to "Pro or Business" reflects the current plan naming convention. This update ensures the migration documentation references the correct tier names for the Commercial Edition. The change is contextually appropriate and improves clarity for users working with paid workspaces.

self-hosting/govern/google-oauth.mdx (1)

28-28: Verify image asset exists before deployment.

Line 28 references a new image file /images/custom-sso/google-oauth.png. This image must exist in the public/images/custom-sso/ directory for proper rendering. The verification shell script above will confirm availability of this and related OAuth configuration images.

self-hosting/govern/oidc-sso.mdx (2)

6-6: LGTM: Plan tier labeling modernized consistently.

The blockquote-formatted plan label now correctly displays "Pro, Business" instead of "Plane One, Plane Pro". This formatting and terminology change aligns with the broader rebranding across the SSO documentation suite and maintains consistency with related files.


8-8: LGTM: Branding text simplified appropriately.

The text change from "Plane One enables custom SSO" to "Plane enables custom SSO" standardizes the branding language across the documentation and removes product-specific references that could become stale.

self-hosting/govern/saml-sso.mdx (2)

6-6: LGTM: Plan tier labeling modernized consistently.

The blockquote-formatted plan label now correctly displays "Pro, Business" instead of "Plane One, Plane Pro", consistent with the broader documentation rebranding across the SSO configuration guides.


8-8: LGTM: Branding text simplified appropriately.

The text change from "Plane One enables custom SSO" to "Plane enables custom SSO" maintains consistency with similar updates across the documentation and removes product-specific references.

@srinivaspendem srinivaspendem merged commit 0122729 into preview Dec 2, 2025
4 of 6 checks passed
@srinivaspendem srinivaspendem deleted the updates-20-11-25 branch December 2, 2025 14:22
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.

3 participants