Send emails as background task

[Mariatta]

Sending emails is time consuming process.
Instead of sending emails directly when triggered in the signals, we should only send the task for sending emails to the background eg to celery.

[codefromlani]

Hi @Mariatta

I'm interested in working on this issue. I've taken a look at the codebase and want to clarify a few things before starting the implementation.

What I Found:
I noticed that sponsorship/signals.py has a post_save signal that sends emails synchronously. Specifically:

sponsorship_profile_signal calls email functions directly in the signal handler

_send_internal_email iterates through multiple recipients (admin/staff users) and sends emails one-by-one in a loop

My Questions:

• Should I add the initial Celery setup as part of this PR? If so, can i use Redis for the message broker, or do you have another preference?

• Should I focus only on the sponsorship/signals.py for now, or are there similar signal-triggered emails in other apps that should also be addressed?

• Is there any existing documentation for running the project with a message broker locally?

[Mariatta]

Hello @codefromlani
Thank you for your interest in this issue. Yes this is still open and you can create the PR.

Should I add the initial Celery setup as part of this PR? If so, can i use Redis for the message broker, or do you have another preference?

Yes you can create the initial celery setup as part of this PR.
We are already using Redis in the project, so let's use that.

Should I focus only on the sponsorship/signals.py for now, or are there similar signal-triggered emails in other apps that should also be addressed?

Eventually all the emails should be triggered as a background task. However, I'm ok with breaking it up into different PRs, eg opening a PR just for the sponsorship-related emails first, and a different PR for volunteer-related emails.

[Mariatta]

Is there any existing documentation for running the project with a message broker locally?

We don't have such documentation yet. If you would like to write the doc for it, it would be appreciated.

[codefromlani]

Thanks for the clarification