Skip to content
This repository has been archived by the owner on May 11, 2021. It is now read-only.

Offload emails to background worker #276

Closed
dirn opened this issue Apr 9, 2016 · 1 comment
Closed

Offload emails to background worker #276

dirn opened this issue Apr 9, 2016 · 1 comment
Assignees

Comments

@dirn
Copy link
Member

dirn commented Apr 9, 2016

There are a several actions that can result in emails being sent. Currently these are all part of Flask-Security, but eventually we'd like to automate some additional ones (#220, #245). If an error occurs sending an email, the user sees an error page. This shouldn't be the case. Instead we should add the emails to a queue and process them through a separate worker. We'd like to use RQ for this.

Since all of the current uses are inside Flask-Security, we can use its send_mail_task decorator to accomplish this.

In addition to adding RQ to the code base, we'll need to add Redis and a worker to the Docker Compose file.

@dirn dirn changed the title Offload emails to task runner Offload emails to background worker Apr 9, 2016
@dirn
Copy link
Member Author

dirn commented Apr 9, 2016

This may not be as straightforward as originally thought. See mattupstate/overholt#33.

@dirn dirn self-assigned this Apr 10, 2016
@dirn dirn closed this as completed Jun 2, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant