Skip to content

feat(notif): clean up notification trait#1673

Merged
seanaye merged 7 commits intomainfrom
seanaye/feat/clean-up-notification-trait
Feb 26, 2026
Merged

feat(notif): clean up notification trait#1673
seanaye merged 7 commits intomainfrom
seanaye/feat/clean-up-notification-trait

Conversation

@seanaye
Copy link
Copy Markdown
Contributor

@seanaye seanaye commented Feb 25, 2026

This PR cleans up the interface for notifications specifically:

  • the safety around how we can construct a QueueMessage.
  • The location of the rate limit configuration (non-optional for email notifications)
  • making things more private if possible
  • add port for digest state machine
  • integrate state machine c with sns worker
  • fix digest deserialization
  • add batch poller to egress
  • update queue message constructor
  • update imports
  • move rate limit into email
  • update visibility
  • fix tests

@seanaye seanaye requested a review from a team as a code owner February 25, 2026 18:36
@seanaye seanaye changed the base branch from main to seanaye/chore/ses-logging February 25, 2026 18:37
@seanaye seanaye changed the base branch from seanaye/chore/ses-logging to seanaye/feat/integrate-state-machine-c February 25, 2026 18:41
@seanaye seanaye force-pushed the seanaye/feat/clean-up-notification-trait branch from 37152f4 to b513369 Compare February 25, 2026 18:44
Base automatically changed from seanaye/feat/integrate-state-machine-c to main February 26, 2026 14:59
@seanaye seanaye force-pushed the seanaye/feat/clean-up-notification-trait branch 2 times, most recently from 671cd80 to f762a8a Compare February 26, 2026 15:50
@seanaye seanaye force-pushed the seanaye/feat/clean-up-notification-trait branch from f762a8a to aeb8f30 Compare February 26, 2026 16:07
@seanaye seanaye merged commit d3406ca into main Feb 26, 2026
39 checks passed
@seanaye seanaye deleted the seanaye/feat/clean-up-notification-trait branch February 26, 2026 19:12
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