chore: Add forum link and remove discord link on readme#8655
chore: Add forum link and remove discord link on readme#8655sriramveeraghanta merged 2 commits intopreviewfrom
Conversation
Removed Discord badge and replaced Releases link with Forum link.
📝 WalkthroughWalkthroughThe README.md file was updated to remove the Discord badge block and the "Releases" link from the header, while adding a new "Forum" link. An empty paragraph was also removed, resulting in a net change of 1 line added and 8 lines removed. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
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.
Pull request overview
Updates the project README’s top navigation/community links to better reflect current community channels.
Changes:
- Removed the Discord badge block from the README header.
- Added a “Forum” link to the top link list.
- Removed the “Releases” link from the top link list.
| <p align="center"> | ||
| <a href="https://plane.so/"><b>Website</b></a> • | ||
| <a href="https://github.com/makeplane/plane/releases"><b>Releases</b></a> • | ||
| <a href="https://forum.plane.so"><b>Forum</b></a> • |
There was a problem hiding this comment.
The commit-activity badge was removed as part of deleting the Discord badge paragraph. If the intent is only to remove Discord, consider keeping the commit-activity badge (move it into the remaining centered paragraph) or document that this PR also removes the badge.
| <p align="center"> | ||
| <a href="https://plane.so/"><b>Website</b></a> • | ||
| <a href="https://github.com/makeplane/plane/releases"><b>Releases</b></a> • | ||
| <a href="https://forum.plane.so"><b>Forum</b></a> • | ||
| <a href="https://twitter.com/planepowers"><b>Twitter</b></a> • |
There was a problem hiding this comment.
The PR title says this change is to add a forum link and remove the Discord link, but this hunk also removes the existing “Releases” link from the top navigation. If removing Releases is intentional, please update the PR title/description; otherwise keep the Releases link and add Forum alongside it.
| <p align="center"> | ||
| <a href="https://plane.so/"><b>Website</b></a> • | ||
| <a href="https://github.com/makeplane/plane/releases"><b>Releases</b></a> • | ||
| <a href="https://forum.plane.so"><b>Forum</b></a> • | ||
| <a href="https://twitter.com/planepowers"><b>Twitter</b></a> • | ||
| <a href="https://docs.plane.so/"><b>Documentation</b></a> | ||
| </p> |
There was a problem hiding this comment.
This PR removes the Discord badge block, but the README still contains multiple Discord links further down (e.g., the “join the conversation on Discord” callouts). If the goal is to remove Discord links from the README, those references should be updated (e.g., to Forum/GitHub Discussions) or the PR title should be narrowed to only the badge removal.
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
README.md (1)
29-29:⚠️ Potential issue | 🟡 MinorResidual Discord links inconsistent with the stated PR intent.
The PR title says "remove discord link on readme," but three inline Discord references remain:
- Line 29 — blockquote directing users to Discord for conversation.
- Line 132 — Community section still mentions "our Discord server."
- Line 148 — Contributing section links
let us knowto the Discord invite URL.If this PR intentionally only removes the Discord badge/header link (keeping body references), that's fine — but the label
[let us know](https://discord.com/invite/A92xrEGCge)on Line 148 is particularly opaque since the Discord destination is not visible to the reader. Consider updating these to point to the Forum or GitHub Discussions, or at minimum making the Discord link label explicit.Also applies to: 132-132, 148-148
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@README.md` at line 29, The README still contains three inline Discord references that contradict the PR intent: the blockquote "Plane is evolving every day..." (blockquote directing users to Discord), the Community section text "our Discord server", and the Contributing link labelled "let us know" that points to the Discord invite; update these occurrences to either (a) replace the Discord invite URL with the Forum or GitHub Discussions URL, or (b) change the link text to explicitly mention "Discord" if you intend to keep it, and preferably add an alternative destination (Forum/GitHub Discussions) in the same places; search for the exact phrases "Plane is evolving every day...", "our Discord server", and the link text "let us know" in README.md and make the replacements consistently.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Outside diff comments:
In `@README.md`:
- Line 29: The README still contains three inline Discord references that
contradict the PR intent: the blockquote "Plane is evolving every day..."
(blockquote directing users to Discord), the Community section text "our Discord
server", and the Contributing link labelled "let us know" that points to the
Discord invite; update these occurrences to either (a) replace the Discord
invite URL with the Forum or GitHub Discussions URL, or (b) change the link text
to explicitly mention "Discord" if you intend to keep it, and preferably add an
alternative destination (Forum/GitHub Discussions) in the same places; search
for the exact phrases "Plane is evolving every day...", "our Discord server",
and the link text "let us know" in README.md and make the replacements
consistently.
Description
Type of Change