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

Fix a number of smaller issues #129

Closed benjaoming closed 10 months ago

benjaoming commented 10 months ago

Using django_nyt in another development project and noticing a number of things here and there.

codecov[bot] commented 10 months ago

Codecov Report

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

Comparison is base (ef7f0d4) 74.90% compared to head (925b0cb) 74.63%.

Files Patch % Lines
django_nyt/models.py 76.36% 13 Missing :warning:
django_nyt/management/commands/notifymail.py 40.00% 3 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #129 +/- ## ========================================== - Coverage 74.90% 74.63% -0.28% ========================================== Files 23 23 Lines 554 611 +57 ========================================== + Hits 415 456 +41 - Misses 139 155 +16 ```

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

benjaoming commented 10 months ago

Thanks for reviewing @oscarmcm :pray: