Skip to content

Conversation

@Adityashandilya555
Copy link
Contributor

@Adityashandilya555 Adityashandilya555 commented Oct 11, 2025

Checklist

  • I have read the OpenWISP Contributing Guidelines.
  • I have manually tested the changes proposed in this pull request.
  • I have written new test cases for new code and/or updated existing tests for changes to existing code.
  • I have updated the documentation.

Reference to Existing Issue

Closes #477 .

Description of Changes

Migrates from archived django-celery-email to maintained django-celery-email-reboot package as suggested
by @nemesifier in #477.
django-celery-email-reboot is a maintained fork that provides Django 4.x/5.x compatibility and is a drop-in
replacement with identical API.

Testing

Followed maintainer's testing approach:

  • Built all Docker images successfully with django-celery-email-reboot 4.1.0
  • Tested email flow: docker compose run dashboard python manage.py sendtestemail [email protected]
  • Verified postfix logs showing successful email pipeline:
    • Django → Celery → Postfix (queue ID 95C4120370E8)
    • Email queued and delivery attempted successfully

Screenshot

Screenshot 2025-10-12 at 2 20 09 AM Screenshot 2025-10-12 at 2 19 20 AM

Adityashandilya555 and others added 5 commits October 4, 2025 09:15
…p#477

Replaced archived django-celery-email with actively maintained
django-post-office for async email handling.

- Updated requirements.txt: django-celery-email -> django-post-office
- Changed email backend configuration in Dockerfile
- Updated settings.py to use post_office app conditionally

Fixes openwisp#477
…penwisp#477

The archived django-celery-email package has been replaced with
django-celery-email-reboot, a maintained fork that provides Django 4.x/5.x
compatibility. This is a drop-in replacement with identical API.

Changes:
- Updated requirements.txt to use django-celery-email-reboot>=4.1.0,<5.0.0
- Changed EMAIL_BACKEND to djcelery_email.backends.CeleryEmailBackend
- Updated settings.py to check for djcelery_email backend

Testing performed following maintainer's approach:
- Built all Docker images successfully with django-celery-email-reboot 4.1.0
- Tested email flow: docker compose run dashboard python manage.py sendtestemail
- Verified postfix logs showing successful email pipeline:
  Django → Celery → Postfix (queue ID 95C4120370E8)
- Email queued and delivery attempted successfully

The entire email pipeline is working correctly with the new package.

Fixes openwisp#477
Copy link
Member

@nemesifier nemesifier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@github-project-automation github-project-automation bot moved this from To do (general) to In progress in OpenWISP Contributor's Board Oct 13, 2025
@github-project-automation github-project-automation bot moved this from In progress to Reviewer approved in OpenWISP Priorities for next releases Oct 13, 2025
@nemesifier nemesifier merged commit a5a6b03 into openwisp:master Oct 13, 2025
1 check passed
@github-project-automation github-project-automation bot moved this from Backlog to Done in 25.09 Release Oct 13, 2025
@github-project-automation github-project-automation bot moved this from In progress to Done in OpenWISP Contributor's Board Oct 13, 2025
@github-project-automation github-project-automation bot moved this from Reviewer approved to Done in OpenWISP Priorities for next releases Oct 13, 2025
@Adityashandilya555 Adityashandilya555 deleted the test-celery-email-reboot branch October 26, 2025 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file enhancement New feature or request

Development

Successfully merging this pull request may close these issues.

[change] Migrate django-celery-email

2 participants