edx-platform icon indicating copy to clipboard operation
edx-platform copied to clipboard

feat: pass required context for instructor emails

Open DmytroAlipov opened this issue 11 months ago • 4 comments

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:

DmytroAlipov avatar Mar 20 '24 19:03 DmytroAlipov

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.

openedx-webhooks avatar Mar 20 '24 19:03 openedx-webhooks

Hi @navinkarkera! I think you're reviewing the master PR for this. Would you mind looking at this backport after?

mphilbrick211 avatar Mar 21 '24 20:03 mphilbrick211

Hi @mphilbrick211 Is it possible to have someone review this MR so that this fix can be included in Quince 3?

DmytroAlipov avatar Apr 08 '24 12:04 DmytroAlipov

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 avatar Apr 10 '24 17:04 mphilbrick211

@mphilbrick211 Yes, once master PR is merged and I'll review and merge this one. cc: @DmytroAlipov

navinkarkera avatar Apr 26 '24 06:04 navinkarkera