Skip to content

Update self-hosted-subscriptions.rst#8850

Merged
esethna merged 2 commits intomasterfrom
wiersgallak-patch-1
Apr 2, 2026
Merged

Update self-hosted-subscriptions.rst#8850
esethna merged 2 commits intomasterfrom
wiersgallak-patch-1

Conversation

@wiersgallak
Copy link
Copy Markdown
Contributor

Summary

Ticket Link

@wiersgallak wiersgallak requested a review from esethna April 2, 2026 15:43
@wiersgallak wiersgallak added the 2: Editor Review Requires review by an editor label Apr 2, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

Newest code from mattermost has been published to preview environment for Git SHA 8812220

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 2, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 9c64ec51-84f8-4251-8569-4b9d4f391d74

📥 Commits

Reviewing files that changed from the base of the PR and between 8812220 and bd49bec.

📒 Files selected for processing (1)
  • source/product-overview/self-hosted-subscriptions.rst
🚧 Files skipped from review as they are similar to previous changes (1)
  • source/product-overview/self-hosted-subscriptions.rst

📝 Walkthrough

Walkthrough

Updated self-hosted subscriptions FAQ: clarified that each license covers a single deployment (one server or an HA group sharing one database), updated non-production license guidance, removed Premier Support tier wording, and specified license key delivery via email after payment.

Changes

Cohort / File(s) Summary
Documentation – Licensing FAQ
source/product-overview/self-hosted-subscriptions.rst
Reworded FAQ to state licenses apply to a single deployment (one server or HA group), added guidance that additional independent servers require separate licenses and to contact account team, replaced Premier Support references with non-production license key guidance (dedicated keys; up to four for Premier customers), and clarified license key is emailed after payment.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description consists only of the repository template with no substantive explanation of the changes made to the self-hosted subscriptions documentation. Provide a meaningful description explaining the FAQ updates regarding license key usage, deployment restrictions, and non-production environment guidance.
Title check ❓ Inconclusive The title is too vague and generic, using only the filename without conveying the substantive changes made to the documentation. Revise the title to describe the actual change, such as 'Clarify self-hosted license deployment and non-production environment requirements' or similar.
✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch wiersgallak-patch-1

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
Copy Markdown
Contributor

@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: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@source/product-overview/self-hosted-subscriptions.rst`:
- Line 98: The sentence "Once your payment is successfully processed your
license key will be provided via email." needs a comma after the introductory
clause; update that sentence to "Once your payment is successfully processed,
your license key will be provided via email." to improve readability.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 2c350c96-ee25-429c-9ce2-a94739e28c47

📥 Commits

Reviewing files that changed from the base of the PR and between 37180a2 and 8812220.

📒 Files selected for processing (1)
  • source/product-overview/self-hosted-subscriptions.rst

Comment thread source/product-overview/self-hosted-subscriptions.rst Outdated
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

Newest code from mattermost has been published to preview environment for Git SHA bd49bec

@esethna esethna merged commit 04e7933 into master Apr 2, 2026
7 checks passed
@esethna esethna deleted the wiersgallak-patch-1 branch April 2, 2026 16:30
@amyblais amyblais removed the 2: Editor Review Requires review by an editor label Apr 7, 2026
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