Skip to content

fix: add /discord redirect on website to fix readme link#12684

Open
spacesitenet wants to merge 12 commits into
Comfy-Org:mainfrom
spacesitenet:fix/discord-page-redirect
Open

fix: add /discord redirect on website to fix readme link#12684
spacesitenet wants to merge 12 commits into
Comfy-Org:mainfrom
spacesitenet:fix/discord-page-redirect

Conversation

@spacesitenet

Copy link
Copy Markdown

Summary

Adds a missing redirect for https://comfy.org/discord to https://discord.com/invite/comfyorg which fixes broken link in this projects readme .

Changes

  • What: Modifies vercel.json to redirect from the discord page to the discord invite. This matches how the blogs redirect is setup.

Review Focus

Worth a check that the destination URL is correct. I've verified the tests related to this /discord page will continue to work. I feel adding this redirect is preferable to changing all links to the discord invite directly to fix any sites already linking to this page.

Screenshots (if applicable)

Screen.Recording.2026-06-06.155511.mp4

@spacesitenet
spacesitenet requested a review from a team June 6, 2026 20:11
@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jun 6, 2026
@coderabbitai

coderabbitai Bot commented Jun 6, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 6d3e80be-4626-40b1-893e-2905e4cb3de2

📥 Commits

Reviewing files that changed from the base of the PR and between 8f314dd and 6854633.

📒 Files selected for processing (1)
  • apps/website/vercel.json

📝 Walkthrough

Walkthrough

This pull request adds a permanent Vercel redirect from /discord to the Discord invite URL.

Changes

Discord Redirect Configuration

Layer / File(s) Summary
Discord redirect rule
apps/website/vercel.json
A permanent redirect maps /discord requests to https://discord.com/invite/comfyorg.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 6
✅ Passed checks (6 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: adding a /discord redirect to fix a broken readme link.
Description check ✅ Passed The description matches the required template with Summary, Changes, Review Focus, and optional Screenshots filled in.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
End-To-End Regression Coverage For Fixes ✅ Passed Changes are only in apps/website/vercel.json, so the browser_tests/ requirement doesn’t apply under this check.
Adr Compliance For Entity/Litegraph Changes ✅ Passed Changed file is only apps/website/vercel.json; no litegraph, ECS, or graph-entity code was modified.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@github-actions

github-actions Bot commented Jun 6, 2026

Copy link
Copy Markdown

🎨 Storybook: loading Building...

@github-actions

github-actions Bot commented Jun 6, 2026

Copy link
Copy Markdown

🎭 Playwright: ⏳ Running...

@spacesitenet

Copy link
Copy Markdown
Author

Fixes #12846

@vercel

vercel Bot commented Jun 18, 2026

Copy link
Copy Markdown

@spacesitenet is attempting to deploy a commit to the Uy Tieu's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

github-actions Bot commented Jun 18, 2026

Copy link
Copy Markdown

🎨 Storybook: 🚧 Building...

🎭 Playwright: ⏳ Running...

@github-actions

github-actions Bot commented Jun 26, 2026

Copy link
Copy Markdown

✅ All contributors have signed the CLA. Thank you! This PR is ready to be merged.
Posted by the CLA Assistant Lite bot.

@spacesitenet

Copy link
Copy Markdown
Author

I have read and agree to the Contributor License Agreement

comfy-legal added a commit to Comfy-Org/comfy-cla that referenced this pull request Jun 26, 2026
@spacesitenet

Copy link
Copy Markdown
Author

Check

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants