search

django-wiki / django-nyt

Notification system for Django with batteries included: Email digests, user settings, JSON API
Apache License 2.0
144 stars 47 forks source link

Adds arguments to set domain and http protocol from CLI #131

Closed benjaoming closed 10 months ago

benjaoming commented 10 months ago

Have an option to avoid using Site.objects.get_current, although we should also check if it's even in INSTALLED_APPS.

codecov[bot] commented 10 months ago

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (925b0cb) 74.63% compared to head (7a9319b) 74.87%. Report is 1 commits behind head on main.

Files Patch % Lines
django_nyt/management/commands/notifymail.py 90.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #131 +/- ## ========================================== + Coverage 74.63% 74.87% +0.24% ========================================== Files 23 23 Lines 611 621 +10 ========================================== + Hits 456 465 +9 - Misses 155 156 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.