Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✅ Files skipped from review due to trivial changes (2)
📝 WalkthroughWalkthroughThis PR updates license management docs and the VitePress sidebar: it collapses four top-level license activation links into two groups ("Commercial Edition" and "Airgapped Edition") and standardizes terminology by replacing "Enterprise" with "Enterprise Grid" across several documentation pages. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
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 |
There was a problem hiding this comment.
Actionable comments posted: 2
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (2)
docs/self-hosting/manage/manage-licenses/activate-airgapped.md (1)
1-7:⚠️ Potential issue | 🟡 MinorFrontmatter title doesn't match H1 heading.
The frontmatter
title(Line 2) is "Activate Airgapped Edition license" but the H1 (Line 7) was updated to "Activate Pro or Business on Airgapped Edition". Update the frontmatter to match the new heading for consistency in navigation and SEO.Proposed fix
--- -title: Activate Airgapped Edition license -description: Activate and configure your Plane Airgapped Edition license. Offline license activation for air-gapped environments without internet access. +title: Activate Pro or Business on Airgapped Edition +description: Activate your Plane Pro or Business license on Airgapped Edition. Offline license activation for air-gapped environments without internet access. keywords: plane airgapped license, offline activation, air-gapped deployment, plane license key, self-hosting ---🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/self-hosting/manage/manage-licenses/activate-airgapped.md` around lines 1 - 7, Update the frontmatter title to match the H1 heading: change the YAML title value "Activate Airgapped Edition license" to "Activate Pro or Business on Airgapped Edition" so the frontmatter `title` and the H1 are identical; locate the YAML `title:` entry at the top of the file and replace its string to exactly match the H1 text.docs/self-hosting/manage/manage-licenses/activate-pro-and-business.md (1)
1-7:⚠️ Potential issue | 🟡 MinorFrontmatter title doesn't match H1 heading.
The frontmatter
title(Line 2) is "Activate Pro and Business licenses" but the H1 (Line 7) was updated to "Activate Pro or Business on Commercial Edition". Update the frontmatter title to match the new heading.Proposed fix
--- -title: Activate Pro and Business licenses +title: Activate Pro or Business on Commercial Edition description: Activate your Plane Pro or Business Edition license. Upgrade from Community Edition to unlock advanced project management features. keywords: plane pro license, plane business license, license activation, commercial edition, plane upgrade, self-hosting ---🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/self-hosting/manage/manage-licenses/activate-pro-and-business.md` around lines 1 - 7, Update the frontmatter title to match the H1 by changing the frontmatter `title` value from "Activate Pro and Business licenses" to "Activate Pro or Business on Commercial Edition"; locate the YAML frontmatter `title:` entry at the top of the file and replace its string so it exactly matches the H1 heading text.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/self-hosting/govern/ldap.md`:
- Around line 15-17: Update the inconsistent terminology: change the instances
of "Enterprise license" (the phrases in the second and third bullet lines) to
"Enterprise Grid license" so all three bullets consistently reference
"Enterprise Grid license" (also update the linked anchor text if it includes the
phrase). Ensure the contact sentence and the "activate" link text match the
updated "Enterprise Grid license" wording.
In `@docs/self-hosting/manage/manage-licenses/activate-enterprise.md`:
- Around line 1-7: Update the frontmatter title value to match the H1 by
changing the title key in the YAML frontmatter (the "title:" line) from
"Activate Enterprise Grid license" to "Activate Enterprise Grid on Commercial
Edition" so the frontmatter and the H1 heading are consistent.
---
Outside diff comments:
In `@docs/self-hosting/manage/manage-licenses/activate-airgapped.md`:
- Around line 1-7: Update the frontmatter title to match the H1 heading: change
the YAML title value "Activate Airgapped Edition license" to "Activate Pro or
Business on Airgapped Edition" so the frontmatter `title` and the H1 are
identical; locate the YAML `title:` entry at the top of the file and replace its
string to exactly match the H1 text.
In `@docs/self-hosting/manage/manage-licenses/activate-pro-and-business.md`:
- Around line 1-7: Update the frontmatter title to match the H1 by changing the
frontmatter `title` value from "Activate Pro and Business licenses" to "Activate
Pro or Business on Commercial Edition"; locate the YAML frontmatter `title:`
entry at the top of the file and replace its string so it exactly matches the H1
heading text.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 324b8d76-b7a0-4fd4-acf9-811ac4cafc83
📒 Files selected for processing (6)
docs/.vitepress/config.mtsdocs/self-hosting/govern/ldap.mddocs/self-hosting/manage/manage-licenses/activate-airgapped-enterprise.mddocs/self-hosting/manage/manage-licenses/activate-airgapped.mddocs/self-hosting/manage/manage-licenses/activate-enterprise.mddocs/self-hosting/manage/manage-licenses/activate-pro-and-business.md
| - Plane Commercial Edition with an active Enterprise Grid license. | ||
| - Don't have an Enterprise license? Contact Sales at [sales@plane.so](mailto:sales@plane.so) to get started. | ||
| - Already have a license? See how to [activate your Enterprise license](/self-hosting/manage/manage-licenses/activate-enterprise). |
There was a problem hiding this comment.
Incomplete terminology update.
Line 15 references "Enterprise Grid license" but Lines 16-17 still use "Enterprise license" without the "Grid" suffix. Update these references for consistency.
Proposed fix
- Plane Commercial Edition with an active Enterprise Grid license.
- - Don't have an Enterprise license? Contact Sales at [sales@plane.so](mailto:sales@plane.so) to get started.
- - Already have a license? See how to [activate your Enterprise license](/self-hosting/manage/manage-licenses/activate-enterprise).
+ - Don't have an Enterprise Grid license? Contact Sales at [sales@plane.so](mailto:sales@plane.so) to get started.
+ - Already have a license? See how to [activate your Enterprise Grid license](/self-hosting/manage/manage-licenses/activate-enterprise).📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| - Plane Commercial Edition with an active Enterprise Grid license. | |
| - Don't have an Enterprise license? Contact Sales at [sales@plane.so](mailto:sales@plane.so) to get started. | |
| - Already have a license? See how to [activate your Enterprise license](/self-hosting/manage/manage-licenses/activate-enterprise). | |
| - Plane Commercial Edition with an active Enterprise Grid license. | |
| - Don't have an Enterprise Grid license? Contact Sales at [sales@plane.so](mailto:sales@plane.so) to get started. | |
| - Already have a license? See how to [activate your Enterprise Grid license](/self-hosting/manage/manage-licenses/activate-enterprise). |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/self-hosting/govern/ldap.md` around lines 15 - 17, Update the
inconsistent terminology: change the instances of "Enterprise license" (the
phrases in the second and third bullet lines) to "Enterprise Grid license" so
all three bullets consistently reference "Enterprise Grid license" (also update
the linked anchor text if it includes the phrase). Ensure the contact sentence
and the "activate" link text match the updated "Enterprise Grid license"
wording.
| --- | ||
| title: Activate Enterprise license | ||
| title: Activate Enterprise Grid license | ||
| description: Activate your Plane Enterprise Edition license. Enable advanced features including SAML SSO, audit logs, and custom roles for your self-hosted instance. | ||
| keywords: plane enterprise license, license activation, enterprise edition, plane enterprise features, self-hosting | ||
| --- | ||
|
|
||
| # Activate Enterprise license | ||
| # Activate Enterprise Grid on Commercial Edition |
There was a problem hiding this comment.
Frontmatter title doesn't match H1 heading.
The frontmatter title (Line 2) is "Activate Enterprise Grid license" but the H1 (Line 7) is "Activate Enterprise Grid on Commercial Edition". Update the frontmatter title to include "on Commercial Edition" for consistency with other pages in this section.
Proposed fix
---
-title: Activate Enterprise Grid license
+title: Activate Enterprise Grid on Commercial Edition
description: Activate your Plane Enterprise Edition license. Enable advanced features including SAML SSO, audit logs, and custom roles for your self-hosted instance.
keywords: plane enterprise license, license activation, enterprise edition, plane enterprise features, self-hosting
---📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| --- | |
| title: Activate Enterprise license | |
| title: Activate Enterprise Grid license | |
| description: Activate your Plane Enterprise Edition license. Enable advanced features including SAML SSO, audit logs, and custom roles for your self-hosted instance. | |
| keywords: plane enterprise license, license activation, enterprise edition, plane enterprise features, self-hosting | |
| --- | |
| # Activate Enterprise license | |
| # Activate Enterprise Grid on Commercial Edition | |
| --- | |
| title: Activate Enterprise Grid on Commercial Edition | |
| description: Activate your Plane Enterprise Edition license. Enable advanced features including SAML SSO, audit logs, and custom roles for your self-hosted instance. | |
| keywords: plane enterprise license, license activation, enterprise edition, plane enterprise features, self-hosting | |
| --- | |
| # Activate Enterprise Grid on Commercial Edition |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/self-hosting/manage/manage-licenses/activate-enterprise.md` around lines
1 - 7, Update the frontmatter title value to match the H1 by changing the title
key in the YAML frontmatter (the "title:" line) from "Activate Enterprise Grid
license" to "Activate Enterprise Grid on Commercial Edition" so the frontmatter
and the H1 heading are consistent.
Description
Type of Change
Screenshots and Media (if applicable)
Test Scenarios
References
Summary by CodeRabbit
Documentation