Skip to content

Use support email - #2889

Closed
nezaj wants to merge 4 commits into
mainfrom
rm-discord
Closed

Use support email#2889
nezaj wants to merge 4 commits into
mainfrom
rm-discord

Conversation

@nezaj

@nezaj nezaj commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Instead directing folks to discord for support questions, let's direct them to our support email

@coderabbitai

coderabbitai Bot commented Aug 31, 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: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: e1c8aab8-a7b2-4cef-b298-d1decf37be89

📥 Commits

Reviewing files that changed from the base of the PR and between 6f6b56a and 0cf6069.

⛔ Files ignored due to path filters (1)
  • client/www/public/marketing/discord-icon.svg is excluded by !**/*.svg
📒 Files selected for processing (68)
  • README.md
  • client/README.md
  • client/packages/admin/README.md
  • client/packages/cli/README.md
  • client/packages/cli/src/old.js
  • client/packages/components/README.md
  • client/packages/components/src/config.ts
  • client/packages/core/README.md
  • client/packages/create-instant-app/README.md
  • client/packages/expo-sqlite/README.md
  • client/packages/mcp/README.md
  • client/packages/platform/README.md
  • client/packages/python/README.md
  • client/packages/react-common/README.md
  • client/packages/react-native-mmkv/README.md
  • client/packages/react-native/README.md
  • client/packages/react/README.md
  • client/packages/resumable-stream/README.md
  • client/packages/solidjs/README.md
  • client/packages/svelte/README.md
  • client/packages/version/README.md
  • client/packages/vue/README.md
  • client/packages/webhooks/README.md
  • client/sandbox/admin-sdk-express/README.md
  • client/sandbox/cli-nodejs/README.md
  • client/sandbox/react-native-expo/README.md
  • client/sandbox/react-nextjs/README.md
  • client/sandbox/strong-init-vite/README.md
  • client/sandbox/sveltekit/README.md
  • client/sandbox/task-tracker/README.md
  • client/sandbox/vanilla-js-vite/README.md
  • client/sandbox/vue-vite/README.md
  • client/www/_posts/architecture.md
  • client/www/app/docs/auth/platform-oauth/page.md
  • client/www/app/docs/devtool/page.md
  • client/www/app/docs/next-ssr/page.md
  • client/www/app/docs/platform-api/page.md
  • client/www/app/docs/presence-and-topics/page.md
  • client/www/app/docs/workflow/page.md
  • client/www/app/tutorial/content.tsx
  • client/www/components/dash/Billing.tsx
  • client/www/components/dash/OAuthApps.tsx
  • client/www/components/dash/Orgs.tsx
  • client/www/components/dash/Perms.tsx
  • client/www/components/dash/auth/Clerk.tsx
  • client/www/components/dash/org-management/OrgBilling.tsx
  • client/www/components/marketingUi.tsx
  • client/www/components/new-landing/Footer.tsx
  • client/www/components/new-landing/icons.tsx
  • client/www/lib/config.ts
  • client/www/next.config.js
  • client/www/pages/_devtool/index.tsx
  • client/www/pages/dash/index.tsx
  • client/www/pages/debug-uri/[trace-id]/[span-id].tsx
  • client/www/pages/platform/oauth/start.tsx
  • examples/bun-react/README.md
  • examples/expo/README.md
  • examples/next-js-app-dir/README.md
  • examples/python-script/README.md
  • examples/solidjs-vite-advanced/README.md
  • examples/tanstack-start-with-tanstack-query/README.md
  • examples/tanstack-start/README.md
  • examples/vite-react/README.md
  • examples/vite-vanilla/README.md
  • server/README.md
  • server/src/instant/db/app_backup_jobs.clj
  • server/src/instant/flags.clj
  • server/src/instant/util/exception.clj
💤 Files with no reviewable changes (5)
  • client/www/next.config.js
  • client/www/components/marketingUi.tsx
  • client/packages/components/src/config.ts
  • client/www/components/new-landing/icons.tsx
  • client/www/lib/config.ts

Included review availability: Your plan provides up to 10 included reviews per hour; 8 remain after this review.


📝 Walkthrough

Walkthrough

The changes remove Discord references across project documentation, package READMEs, examples, website UI, redirects, configuration exports, and server messages. Support directions now use email, GitHub, or issue and pull request workflows.

Changes

Documentation contact updates

Layer / File(s) Summary
Repository, package, sandbox, and example documentation
README.md, client/**/README.md, examples/**/README.md, server/README.md
Discord badges, navigation links, and support instructions were removed or replaced with support email, GitHub, or issue and pull request links.
Website documentation
client/www/_posts/architecture.md, client/www/app/docs/**, client/www/app/tutorial/content.tsx
Documentation support links now use email or GitHub. A Discord-specific chat example was generalized.

Application support paths

Layer / File(s) Summary
Support messages, footer links, and configuration
client/packages/cli/src/old.js, client/www/components/**, client/www/lib/config.ts, client/www/next.config.js
Application messages and footer links no longer direct users to Discord. Removed Discord configuration exports, the Discord icon, and the /discord redirect.
Dashboard and platform flows
client/www/pages/**
Dashboard, devtool, debug URI, and OAuth messages now provide support email links. The community action now links to GitHub and tracks view_github.

Server support messages

Layer / File(s) Summary
Backup and rate-limit messaging
server/src/instant/db/app_backup_jobs.clj, server/src/instant/flags.clj, server/src/instant/util/exception.clj
Server messages and documentation now direct users to support email instead of Discord. The unused backup invite constant was removed.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to 0cf60

This PR redirects support messaging from Discord to email or issue-based channels without changing application behavior or interfaces. No actionable merge-blocking risk remains beyond normal checks.

Suggested reviewers: dwwoelfel

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 5.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 20 functions across 13 files. (50 skipped:… Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely identifies the main change: directing users to support email instead of Discord.
Description check ✅ Passed The description accurately summarizes the project-wide change from Discord support guidance to the support email.
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.
Full details: Docstring Coverage

Explanation

Docstring coverage is 5.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 20 functions across 13 files. (50 skipped: 50 unsupported.)

  • Fix all pre-merge checks with AI

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.

@dwwoelfel dwwoelfel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@github-actions

Copy link
Copy Markdown
Contributor

View Vercel preview at instant-www-js-rm-discord-jsv.vercel.app.

@nezaj nezaj closed this Aug 31, 2026
@nezaj

nezaj commented Aug 31, 2026

Copy link
Copy Markdown
Contributor Author

Closing for now and will reconsider later!

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.

2 participants