Skip to content

Backend/emails: Port to host request emails to new system#8714

Merged
tristanlabelle merged 5 commits into
developfrom
backend/refactor/emails-new-system-part-3
May 26, 2026
Merged

Backend/emails: Port to host request emails to new system#8714
tristanlabelle merged 5 commits into
developfrom
backend/refactor/emails-new-system-part-3

Conversation

@tristanlabelle

@tristanlabelle tristanlabelle commented May 20, 2026

Copy link
Copy Markdown
Contributor

Ports host request emails to the new email system.

I added a system to share strings between different emails. It's not the prettiest but I intend to iterate on it once I'm done porting emails.

Depends on #8715 to get the side-by-side layout for the "View request" and "Quick decline" buttons. Without it the buttons will be one on top of each other.

Testing

Inspected emails resulting from uv run dump-emails.

(See note above about stacked buttons)

image image image

Backend checklist

  • Added tests for any new code or added a regression test if fixing a bug
  • Run the backend locally and it works
  • Added migrations if there are any database changes, rebased onto develop if necessary for linear migration history

For maintainers

  • Maintainers can push commits to my branch
  • Maintainers can merge this PR for me

@vercel

vercel Bot commented May 20, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
couchers Ready Ready Preview May 25, 2026 12:30am

Request Review

@tristanlabelle
tristanlabelle force-pushed the backend/refactor/emails-new-system-part-3 branch from 3779ed1 to 35e0179 Compare May 20, 2026 08:28
@tristanlabelle
tristanlabelle marked this pull request as ready for review May 20, 2026 08:36
@tristanlabelle
tristanlabelle requested a review from aapeliv as a code owner May 20, 2026 08:36
@tristanlabelle
tristanlabelle requested a review from a team May 20, 2026 08:36
@tristanlabelle tristanlabelle changed the title Backend/emails: Port to new email system (host requests) Backend/emails: Port to host request emails to new system May 20, 2026
@tristanlabelle
tristanlabelle merged commit 3aab94d into develop May 26, 2026
6 checks passed
@tristanlabelle
tristanlabelle deleted the backend/refactor/emails-new-system-part-3 branch May 26, 2026 17:00
@github-actions

Copy link
Copy Markdown
Contributor

📝 Release Notes

This PR does not need to be included in release notes.

Reason: This PR mainly ports existing host request emails to a new internal email system without adding a clearly new user-facing feature or fixing a major user-visible problem. Although users may notice some email layout consistency and the quick decline button remains supported, this is primarily backend/template migration work.

🤖 Bot Debug Information

Model: couchers.openai.gpt-5.4
Decision: exclude
Reasoning: This PR mainly ports existing host request emails to a new internal email system without adding a clearly new user-facing feature or fixing a major user-visible problem. Although users may notice some email layout consistency and the quick decline button remains supported, this is primarily backend/template migration work.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants