edx-platform
edx-platform copied to clipboard
feat: pass required context for instructor emails
This is a backport from the master branch.
Pass required context to bulk enrollment emails:
- logo_url
- homepage_url
- dashboard_url
Add additional context for enrollment emails:
- contact_email
- platform_name
There is no required context for openedx/core/djangoapps/ace_common/templates/ace_common/edx_ace/common/base_body.html:
Thanks for the pull request, @DmytroAlipov! Please note that it may take us up to several weeks or months to complete a review and merge your PR.
Feel free to add as much of the following information to the ticket as you can:
- supporting documentation
- Open edX discussion forum threads
- timeline information ("this must be merged by XX date", and why that is)
- partner information ("this is a course on edx.org")
- any other information that can help Product understand the context for the PR
All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.
Please let us know once your PR is ready for our review and all tests are green.
Hi @navinkarkera! I think you're reviewing the master PR for this. Would you mind looking at this backport after?
Hi @mphilbrick211 Is it possible to have someone review this MR so that this fix can be included in Quince 3?
Hi @mphilbrick211 Is it possible to have someone review this MR so that this fix can be included in Quince 3?
Hi @navinkarkera! Following up on this to see if you could review?
@mphilbrick211 Yes, once master PR is merged and I'll review and merge this one. cc: @DmytroAlipov