Skip to content
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.

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

feat: add send email command for case 1 #90

Open
wants to merge 12 commits into
base: and/temporal_payment_notifications
Choose a base branch
from

Conversation

johanseto
Copy link
Collaborator

@johanseto johanseto commented Aug 17, 2023

Description

Add command to send emails case 1

email template used

https://app.slack.com/client/T04QEEPSM8U/C04Q5CSKA8N/thread/C04Q5CSKA8N-1692193164.102439

Testing instructions

After

Tested for case 1 using debug server.
2023-08-17_16-21
Peek 2023-08-17 16-19
image

Additional information

Payment notifications...

Checklist for Merge

  • Tested in a remote environment
  • Updated documentation
  • Rebased master/main
  • Squashed commits

@github-actions github-actions bot added commands size/m m lines label labels Aug 17, 2023
@johanseto johanseto self-assigned this Aug 17, 2023
@johanseto johanseto force-pushed the jlc/payment-notifications-emails branch from 596cd16 to b752913 Compare August 17, 2023 21:50
chore: first iteration

feat: add more logic to send summary email

fix: get objects and logger erro

feat: add course display_name and arabic name
@johanseto johanseto force-pushed the jlc/payment-notifications-emails branch from b752913 to d9d468c Compare August 17, 2023 21:52
@johanseto johanseto force-pushed the jlc/payment-notifications-emails branch from 3845b8b to a4f21df Compare August 18, 2023 20:55
@johanseto johanseto force-pushed the jlc/payment-notifications-emails branch from 9d029fa to d19f326 Compare August 18, 2023 21:31
@andrey-canon andrey-canon force-pushed the and/temporal_payment_notifications branch 4 times, most recently from 9f1aff5 to c0fe7e3 Compare September 8, 2023 22:03
@johanseto johanseto force-pushed the and/temporal_payment_notifications branch from c0fe7e3 to 3135124 Compare September 13, 2023 21:28
@andrey-canon andrey-canon force-pushed the and/temporal_payment_notifications branch from 3135124 to 39452b7 Compare September 14, 2023 21:17
@Ever3tt Ever3tt force-pushed the and/temporal_payment_notifications branch 2 times, most recently from 44e3736 to 68c122b Compare October 2, 2023 14:47
@andrey-canon andrey-canon force-pushed the and/temporal_payment_notifications branch from abd773f to 4022c03 Compare October 3, 2023 22:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
commands size/m m lines label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants