Skip to content

Added global_vars to MarkdownMail rendering, removed redundant contexts #4647

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
wes-otf merged 1 commit into from
Jan 9, 2026
Merged

Added global_vars to MarkdownMail rendering, removed redundant contexts #4647

wes-otf merged 1 commit into from
Jan 9, 2026
+19 −40

Conversation

@wes-otf
Copy link
Contributor

@wes-otf wes-otf commented Dec 8, 2025

…MarkdownMail renderer

Relates to #4638 - removes the redundant markdown email contexts in favor of just utilizing the global_vars context processor

Test Steps

  • Ensure all markdown emails still look as they did before

@wes-otf wes-otf added Type: Enhancement This is an improvement of an existing thing (not a new thing, which would be a feature). Type: Patch Mini change, used in release drafter labels Dec 8, 2025
@wes-otf wes-otf added Status: Needs dev testing 🧑‍💻 Tasks that should be tested by the dev team Status: Tested - approved for live ✅ and removed Status: Needs dev testing 🧑‍💻 Tasks that should be tested by the dev team labels Jan 9, 2026
@wes-otf wes-otf merged commit a782e8b into main Jan 9, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Status: Tested - approved for live ✅ Type: Enhancement This is an improvement of an existing thing (not a new thing, which would be a feature). Type: Patch Mini change, used in release drafter

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@wes-otf