Skip to content

Conversation

@waleedlatif1
Copy link
Collaborator

Summary

Type of Change

  • Bug fix

Testing

N/A

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • [x I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel
Copy link

vercel bot commented Jan 15, 2026

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

1 Skipped Deployment
Project Deployment Review Updated (UTC)
docs Skipped Skipped Jan 15, 2026 2:51am

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 15, 2026

Greptile Summary

Updated email footer social media links to use local redirects (sim.ai/discord, sim.ai/x, sim.ai/github) instead of direct external URLs to improve email deliverability.

Changes:

  • Modified email-footer.tsx to route X, Discord, and GitHub links through baseUrl redirects
  • Added three new redirect rules in next.config.ts with permanent: false for temporary redirects

Rationale:
According to Resend's deliverability insights, having email links share the same base URL improves deliverability by reducing the number of external domains in emails.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • The changes are straightforward and well-justified: email footer links now route through local redirects instead of direct external URLs. The implementation is clean with proper use of temporary redirects (permanent: false) for social media links that may change. No logical errors, security issues, or breaking changes introduced.
  • No files require special attention

Important Files Changed

Filename Overview
apps/sim/components/emails/components/email-footer.tsx Updated social media links to use baseUrl redirects instead of direct external URLs
apps/sim/next.config.ts Added redirect rules for /discord, /x, and /github paths with permanent: false

Sequence Diagram

sequenceDiagram
    participant User
    participant Email
    participant sim.ai
    participant External
    
    Note over Email: Email footer with social links
    User->>Email: Click social media link (e.g., Discord)
    Email->>sim.ai: GET sim.ai/discord
    Note over sim.ai: Next.js redirect rule<br/>permanent: false
    sim.ai->>External: 302 Redirect to discord.gg/Hr4UWYEcTT
    External->>User: Load Discord invite page
    
    Note over sim.ai,External: Same pattern for /x → x.com/simdotai<br/>and /github → github.com/simstudioai/sim
Loading

@waleedlatif1 waleedlatif1 merged commit 3db9ad2 into staging Jan 15, 2026
7 checks passed
@waleedlatif1 waleedlatif1 deleted the improvement/emails branch January 15, 2026 02:53
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