Skip to content
This repository has been archived by the owner on Nov 4, 2024. It is now read-only.

Enterprise offer usage email updates #170

Conversation

iloveagent57
Copy link
Contributor

@iloveagent57 iloveagent57 commented Jun 27, 2022

  • Install edx-braze-client and create a new EdxBrazeClient class with the library client class as its base.
  • Use the new client for sending API-triggered campaign messages for the Enterprise Offer Usage email.
  • The send_offer_usage_email_via_braze task now takes a dict of {email: lms_user_id} instead of a simple list of email addresses as its first parameter.
  • Attempt to create_recipient with the lms user ID as the external ID for each email to which this message is sent.

Used by: #169

Merge checklist:

  • Any new requirements are in the right place (do not manually modify the requirements/*.txt files)
    • base.in if needed in production
    • test.in for test requirements
    • make upgrade && make requirements have been run to regenerate requirements
  • Version bumped

Post merge:

  • Tag pushed and a new version released
    • Note: Assets will be added automatically. You just need to provide a tag (should match your version number) and title and description.
  • After versioned build finishes in GitHub CI, verify version has been pushed to PyPI
    • Each step in the release build has a condition flag that checks if the rest of the steps are done and if so will deploy to PyPi.
      (so basically once your build finishes, after maybe a minute you should see the new version in PyPi automatically (on refresh))
  • PR created in ecommerce to upgrade dependencies (including ecommerce-worker)
    • This must be done after the version is visible in PyPi as make upgrade in ecommerce will look for the latest version in PyPi.
    • Note: the ecommerce-worker constraint in ecommerce must also be bumped to the latest version in PyPi.
  • Deploy ecommerce
  • Deploy ecomworker on GoCD.
    • While some functions in ecommerce-worker are run via ecommerce, others are handled by a standalone AMI and must be
      released via GoCD.

@iloveagent57 iloveagent57 force-pushed the aed/enterprise-offer-usage-email branch 6 times, most recently from d25597a to b364774 Compare June 27, 2022 17:25
ecommerce_worker/email/v1/braze/tasks.py Outdated Show resolved Hide resolved
ecommerce_worker/email/v1/tasks.py Outdated Show resolved Hide resolved
@iloveagent57 iloveagent57 force-pushed the aed/enterprise-offer-usage-email branch 5 times, most recently from 8853d54 to 0289ef1 Compare June 28, 2022 15:34
* Install edx-braze-client and create a new EdxBrazeClient class with the library client class as its base.
* Use the new client for sending API-triggered campaign messages for the Enterprise Offer Usage email.
* The send_offer_usage_email_via_braze task now takes a dict of {email: lms_user_id} instead of a simple list of email addresses as its first parameter.
* Attempt to create_recipient with the lms user ID as the external ID for each email to which this message is sent.
ENT-5940
@iloveagent57 iloveagent57 force-pushed the aed/enterprise-offer-usage-email branch from 0289ef1 to 676cec8 Compare June 28, 2022 15:47
@iloveagent57 iloveagent57 merged commit 4f94680 into openedx-unsupported:master Jun 29, 2022
@iloveagent57 iloveagent57 deleted the aed/enterprise-offer-usage-email branch June 29, 2022 13:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants