Email Async Backend - #82
Merged
Merged
Conversation
Member
Author
This stack of pull requests is managed by Graphite. Learn more about stacking. |
isabellalam12
changed the base branch from
email-async-frontend
to
graphite-base/82
August 17, 2026 21:03
isabellalam12
force-pushed
the
email-async-backend
branch
from
August 17, 2026 21:03
114d7ce to
ce53f24
Compare
isabellalam12
force-pushed
the
graphite-base/82
branch
from
August 17, 2026 21:03
75f0444 to
6e2e2a1
Compare
isabellalam12
force-pushed
the
email-async-backend
branch
from
August 17, 2026 21:06
ce53f24 to
56fc6a4
Compare
Plan and implement asynchronous email backend and UI. Swap to database-backed templates. Fix backend and frontent tests Remove MatchingSendPage and dedupe sending
isabellalam12
force-pushed
the
email-async-backend
branch
from
August 17, 2026 21:09
56fc6a4 to
4fce3f3
Compare
isabellalam12
force-pushed
the
email-async-backend
branch
from
August 19, 2026 15:32
d14ca7a to
ebd603a
Compare
|
![graphite-app[bot]](https://avatars.githubusercontent.com/in/158384?s=60&v=4){kind=small}
Graphite Automations"Request reviewers once CI passes" took an action on this PR • (08/19/26)2 reviewers were added to this PR based on Henry Chen's automation. |
arklian
approved these changes
Aug 21, 2026
Member
Author
Merge activity
|
spiffyy99
reviewed
Aug 21, 2026
| } | ||
|
|
||
| // Async email flow with template ID (replaces SendRequest for new code) | ||
| export interface SendAsyncRequest { |
There was a problem hiding this comment.
Note here that this can be used for both the legacy (non async) and new async paths.
We may want to add the ability in the future to send emails without a template (adhoc custom message) but saving everything as a template is fine for now.
| @Slf4j | ||
| public class EmailDrainer { | ||
|
|
||
| private static final int BATCH_SIZE = 50; |
There was a problem hiding this comment.
Nit for the future - ideally make something like this configurable based on our pairings size.
Can live in a conf file/env var so we can tweak it easily without redeploying the code
Allimonae
pushed a commit
that referenced
this pull request
Aug 21, 2026
Plan and implement asynchronous email backend and UI. Swap to database-backed templates. Fix backend and frontent tests Remove MatchingSendPage and dedupe sending
This was referenced Aug 24, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Plan and implement asynchronous email backend and UI. Swap to database-backed templates.
Fix backend and frontent tests
Remove MatchingSendPage and dedupe sending