Feat/ses enrollment email templates#259
Draft
nthriveni-sonata-ship-it wants to merge 5 commits into
Draft
Conversation
7c24e49 to
3c10a22
Compare
…(no behavior change)
3c10a22 to
d51c9dd
Compare
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
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.
Context
Course enrollment emails are currently sent via Braze Canvas, where:
canvas_entry_propertiesWhat this PR contains
This PR introduces draft Django email templates for course enrollment:
English and Spanish templates
Header and footer partials
Supporting static assets added under:
Initial backend scaffolding in
tasks.pyto support future SES renderingThese templates are designed to replicate the current Braze email design and behavior.
Important — Review-Only PR
This PR is not intended to be merged.
It is shared strictly for:
There are no functional changes and no impact on the current email flow.
Braze remains the only active rendering mechanism.
Backend Notes (Scope Clarification)
tasks.pyincludes preparatory changes onlyThese changes are incomplete by design and not wired into any execution path.
tasks.pyis not part of the review scope for this PR.Why templates are in
edx-platform(and notedx-themes)These templates are intentionally added to
edx-platformfor this review.Reason:
Adding them to
edx-themesat this stage would:Intentional placement
Placing them in
edx-platform:What comes next
After review and alignment:
Templates will be refined based on feedback
SES-based rendering will be implemented
The flow will be gated via a waffle flag:
Once templates are part of the active rendering flow, theming considerations (including
edx-themes) can be revisitedTesting
Templates have been validated locally for:
No runtime behavior changes are introduced.
Summary
This PR serves as a design and structure review checkpoint:
edx-platformas non-active, review-only artifacts