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

As a developer, I'd like to use a task queue to handle sending notification emails #1952

Closed
5 tasks
riatzukiza opened this issue Jul 13, 2022 · 0 comments · Fixed by #2129
Closed
5 tasks
Assignees
Labels
dev raft review This issue is ready for raft review Refined Ticket has been refined at the backlog refinement

Comments

@riatzukiza
Copy link

riatzukiza commented Jul 13, 2022

Description:

Follow on to #1784
After validating the network settings allowing us to send emails from cloud.gov, we decided not to implement it as a django view, but in a celery task.

Will require functioning celery service to implement this ticket correctly

Acceptance Criteria:
Create a list of functional outcomes that must be achieved to complete this issue

  • Emails are sent from a task queue asynchronously and do not block our django process
  • Testing Checklist has been run and all tests pass
  • README is updated, if necessary

Tasks:
Create a list of granular, specific work items that must be completed to deliver the desired outcomes of this issue

  • Create a class/set of functions for sending emails through the task queue
  • Run Testing Checklist and confirm all tests pass

Notes:
Add additional useful information, such as related issues and functionality that isn't covered by this specific issue, and other considerations that will be helpful for anyone reading this

Supporting Documentation:

https://docs.djangoproject.com/en/4.0/topics/email/
https://docs.djangoproject.com/en/4.0/ref/settings/#std-setting-EMAIL_USE_SSL
https://docs.djangoproject.com/en/dev/ref/templates/api/#loading-a-template

from django.core.mail import send_mail

send_mail(
    'Subject here',
    'Here is the message.',
    'from@example.com',
    ['to@example.com'],
    fail_silently=False,
)

Open Questions:
Please include any questions or decisions that must be made before beginning work or to confidently call this issue complete

  • Open Question 1
  • Open Question 2
@riatzukiza riatzukiza added the dev label Jul 13, 2022
@stevenino stevenino added the Refined Ticket has been refined at the backlog refinement label Aug 11, 2022
@andrew-jameson andrew-jameson changed the title As a developer I'd like to use a task queue to handle sending notification emails As a developer, I'd like to use a task queue to handle sending notification emails Aug 16, 2022
@n0remac n0remac added the raft review This issue is ready for raft review label Sep 16, 2022
@n0remac n0remac linked a pull request Sep 16, 2022 that will close this issue
30 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev raft review This issue is ready for raft review Refined Ticket has been refined at the backlog refinement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants