Skip to content

Fixes from Support threads#146

Merged
danciaclara merged 4 commits intopreviewfrom
fixes-19-09-2025
Sep 18, 2025
Merged

Fixes from Support threads#146
danciaclara merged 4 commits intopreviewfrom
fixes-19-09-2025

Conversation

@danciaclara
Copy link
Collaborator

@danciaclara danciaclara commented Sep 18, 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

Summary by CodeRabbit

  • Documentation
    • Reverse proxy: added a step to restart the instance after editing plane.env (sudo prime-cli restart).
    • Backup/restore: updated the restore script download URL to the preview release path.
    • Kubernetes: guidance changed to use the latest Helm chart version instead of pinning v1.3.0.
    • Podman Quadlets: added a curl command to download the release tarball with a placeholder.
    • Upgrade from Community: simplified wording for copying backup files between servers.

@vercel
Copy link

vercel bot commented Sep 18, 2025

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

Project Deployment Preview Comments Updated (UTC)
developer-docs Error Error Sep 18, 2025 7:54am

@coderabbitai
Copy link

coderabbitai bot commented Sep 18, 2025

Note

Other AI code review bot(s) detected

CodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review.

Walkthrough

Documentation updates across self-hosting guides: added an instance restart step in reverse-proxy setup, updated the restore script download URL, changed a Helm chart note to recommend the latest version, added a curl download step for Podman Quadlets, and clarified wording in the upgrade-from-community instructions.

Changes

Cohort / File(s) Summary of Changes
Reverse Proxy Setup
self-hosting/govern/reverse-proxy.mdx
Inserted a step to restart the instance after editing plane.env with the command sudo prime-cli restart.
Backup & Restore
self-hosting/manage/backup-restore.mdx
Updated restore script download URL from .../master/deploy/selfhost/restore.sh to .../refs/heads/preview/deployments/cli/community/restore.sh.
Upgrade From Community
self-hosting/upgrade-from-community.mdx
Minor wording change: expanded "3" to "three" and removed "on any folder of your choice" from the copy instruction.
Kubernetes & Podman Methods
self-hosting/methods/kubernetes.mdx, self-hosting/methods/podman-quadlets.mdx
Kubernetes: replaced pinned Helm chart version note (v1.3.0) with instruction to use the latest Helm chart version. Podman: added curl -fsSL https://prime.plane.so/releases/<plane-version>/podman-quadlets.tar.gz -o podman-quadlets.tar.gz before extraction.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • sriramveeraghanta
  • mguptahub
  • aheckmann

Poem

A rabbit taps keys with a gentle hop,
Restart, curl, and versions that don’t stop.
Backups routed, upgrades neat,
Kubernetes notes now light on their feet.
Docs refreshed — we thump, then hop! 🐇✨

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title Check ❓ Inconclusive The title "Fixes from Support threads" correctly signals that multiple fixes are included but is too generic to convey the primary change(s). The changeset contains several distinct documentation edits (plane.env restart step, updated restore script URL, Helm chart note, Podman Quadlets download command, and a minor wording tweak) and the PR body is just the template, so the title does not give reviewers a clear, scannable summary of the main change. Please update the PR title to a concise, descriptive summary of the main change (for example: "Docs: add plane.env restart step; update restore script URL and Podman Quadlets download") and expand the PR description to list the files changed and the intent of each edit so reviewers can assess scope quickly. If there is a single primary fix, make the title focus on that and mention the other minor fixes in the body. If these are unrelated fixes, consider splitting them into separate PRs to simplify review.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fixes-19-09-2025

📜 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 edd24a7 and 1ae9c88.

📒 Files selected for processing (3)
  • self-hosting/methods/kubernetes.mdx (1 hunks)
  • self-hosting/methods/podman-quadlets.mdx (1 hunks)
  • self-hosting/upgrade-from-community.mdx (1 hunks)
✅ Files skipped from review due to trivial changes (1)
  • self-hosting/methods/kubernetes.mdx
🚧 Files skipped from review as they are similar to previous changes (2)
  • self-hosting/methods/podman-quadlets.mdx
  • self-hosting/upgrade-from-community.mdx

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.

@danciaclara danciaclara requested review from akshat5302 and sriramveeraghanta and removed request for sriramveeraghanta September 18, 2025 07:45
cursor[bot]

This comment was marked as outdated.

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

🧹 Nitpick comments (2)
self-hosting/methods/kubernetes.mdx (1)

20-20: Fix typo in Note.

Duplicate "use." Recommend tightening the sentence.

-Ensure you're using use the latest Helm chart version.
+Ensure you use the latest Helm chart version.
self-hosting/upgrade-from-community.mdx (1)

36-36: Grammar: “all three files.”

Small polish for clarity.

-4. Copy all the 3 files from the server running the Community Edition to any folder on the server running the Commercial Edition.
+4. Copy all three files from the server running the Community Edition to any folder on the server running the Commercial Edition.
📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9f5b268 and edd24a7.

📒 Files selected for processing (5)
  • self-hosting/govern/reverse-proxy.mdx (1 hunks)
  • self-hosting/manage/backup-restore.mdx (1 hunks)
  • self-hosting/methods/kubernetes.mdx (1 hunks)
  • self-hosting/methods/podman-quadlets.mdx (1 hunks)
  • self-hosting/upgrade-from-community.mdx (1 hunks)
🔇 Additional comments (2)
self-hosting/govern/reverse-proxy.mdx (1)

24-27: Good addition—explicit restart after env changes.

self-hosting/manage/backup-restore.mdx (1)

88-91: Original concern incorrect — both raw.githubusercontent.com URLs resolve; no change required. Both the refs/heads/preview and preview forms returned HTTP/2 200 when tested.

Likely an incorrect or invalid review comment.

@danciaclara danciaclara merged commit fa157ba into preview Sep 18, 2025
5 of 6 checks passed
@danciaclara danciaclara deleted the fixes-19-09-2025 branch September 18, 2025 08:00
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