Skip to content

Comments

Consolidate email templating#7490

Draft
tristanlabelle wants to merge 5 commits intodevelopfrom
backend/refactor/consolidate-email-templating
Draft

Consolidate email templating#7490
tristanlabelle wants to merge 5 commits intodevelopfrom
backend/refactor/consolidate-email-templating

Conversation

@tristanlabelle
Copy link
Contributor

Describe briefly what this PR is doing and why

Please reference any issues this closes with closes #[issue number].

Please give clear steps for how the reviewer can best test this PR

Please include any necessary dev environment, .env, etc. adjustments.

Backend checklist

  • Formatted my code by running make format in app/backend
  • Added tests for any new code or added a regression test if fixing a bug
  • All tests pass
  • Run the backend locally and it works
  • Added migrations if there are any database changes, rebased onto develop if necessary for linear migration history

Web frontend checklist

  • Formatted my code with yarn format
  • There are no warnings from yarn lint --fix
  • There are no console warnings when running the app
  • Added tests where relevant
  • All tests pass
  • Clicked around my changes running locally and it works
  • Checked Desktop, Mobile and Tablet screen sizes

Other
Untick the following if you'd prefer that maintainers don't push commits/merge your branch.

  • A maintainer can push commits to my branch
  • A maintainer can merge my PR (you can also merge after approval)

@vercel
Copy link

vercel bot commented Dec 21, 2025

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

Project Deployment Review Updated (UTC)
couchers Ready Ready Preview Dec 23, 2025 9:57am

@nabramow
Copy link
Member

Hey @tristanlabelle you still working on this? Doing some cleanup on the boards, etc. and wanted to check.

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