Skip to content

Conversation

@mastercactapus
Copy link
Member

Description:
Adds a message_status_history table to track the history of messages over time.

  • Cleaned up along with outgoing_messages via foreign key
  • Alert details page updated to allow viewing full message history

Out of Scope:

  • Admin message logs has not been updated to display the new history.
  • No metrics are emitted or aggregated for things like time-to-delivery.
  • Alert log messages are added when sent but not when initially scheduled, this change makes fixing that a natural improvement, but is not in this PR.

Screenshots:
image

Describe any introduced user-facing changes:

  • Alert logs now have an expansion option for sent notifications

Describe any introduced API changes:
New query for getting the history of a message.

Copy link
Collaborator

@AlaricWhitney AlaricWhitney left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mastercactapus mastercactapus merged commit 611d313 into master Mar 28, 2025
9 checks passed
@mastercactapus mastercactapus deleted the message-status-history branch March 28, 2025 15:21
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.

4 participants