Skip to content

Strip angle brackets - #3825

Merged
dsubak merged 1 commit into
mainfrom
dansubak/202608_strip_message_id
Aug 6, 2026
Merged

Strip angle brackets#3825
dsubak merged 1 commit into
mainfrom
dansubak/202608_strip_message_id

Conversation

@dsubak

@dsubak dsubak commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

What are the relevant tickets?

https://mit-office-of-digital-learning.sentry.io/issues/7656349183/events/394cbee32c4048f3a318b1b3c6cf8dd8/?environment=rc&project=5864687&query=is%3Aunresolved&referrer=previous-event

Description (What does it do?)

Just strips some formatting from the message ID given back by anymail. This means that it'll match what we get back from mailgun in the webhooks.

If you're using something like mailpit locally, this change has no effect.

How can this be tested?

Tests should pass - this is otherwise not feasible to test w/o a bunch of finagling and a mailgun account.

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

OpenAPI Changes

Show/hide changes
## Changes for v0.yaml:
No changes detected

## Changes for v1.yaml:
No changes detected

## Changes for v2.yaml:
No changes detected

Unexpected changes? Ensure your branch is up-to-date with main (consider rebasing).

@dsubak
dsubak merged commit 919ad0c into main Aug 6, 2026
13 checks passed
@dsubak
dsubak deleted the dansubak/202608_strip_message_id branch August 6, 2026 14:20
@odlbot odlbot mentioned this pull request Aug 6, 2026
3 tasks
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