Remove welcome email

[arjunkomath]

Summary by CodeRabbit

• Chores
  • Removed the automated welcome email notification sent to users upon account creation.

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
changes-page	Ready	Preview	Feb 10, 2026 7:16am

2 Skipped Deployments

Project	Deployment	Actions	Updated (UTC)
changes-page-docs	Skipped		Feb 10, 2026 7:16am
user-changes-page	Skipped		Feb 10, 2026 7:16am

[coderabbitai]

📝 Walkthrough

Walkthrough

The changes remove the welcome email functionality by deleting the Inngest-based email handler function, removing its import from the API route handler, and eliminating the event trigger from the webhook processor that previously dispatched welcome emails upon user creation.

Changes

Cohort / File(s)	Summary
Inngest Email Function Removal apps/web/inngest/email/send-welcome-email.ts	Deleted entire file containing the welcome email Inngest function that previously listened for email/user.welcome events and sent templated emails via Postmark.
API Route Integration Cleanup apps/web/pages/api/inngest.ts, apps/web/pages/api/users/webhook.ts	Removed sendWelcomeEmail import and function reference from Inngest serve handler; removed welcome email event emission logic from webhook processor on user INSERT operations, replacing strongly-typed user object extraction with looser record variable handling.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~15 minutes

Poem

🐰 A welcome once sent through the digital breeze,
Now hops off to silence with elegant ease.
No templates to whisper, no emails to fly—
The rabbit just logs and waves them goodbye! 📧✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The pull request title 'Remove welcome email' directly and clearly describes the main change—deletion of welcome email functionality across three files.
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 docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch develop

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[arjunkomath]

@coderabbitai review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@apps/web/pages/api/users/webhook.ts`:
- Around line 13-14: The console.log accesses user.id without ensuring a record
exists; guard against missing payload by checking record || old_record before
using it: in the webhook handler where you compute const user = record ||
old_record and call console.log("Trigger databaseWebhook [Users]: Record:",
type, user.id), first verify user is defined (or fallback to a safe placeholder)
and only access user.id when present, otherwise log a clear message and return
or handle the empty payload gracefully to avoid a TypeError and prevent handler
crashes.