Fix 265 - set up logging + log emails - Githubissues

LesFruitsDefendus / saskatoon-ng

The new generation Saskatoon harverst management system

GNU Affero General Public License v3.0

10 stars 24 forks source link

Fix 265 - set up logging + log emails #363

Closed 2ynn closed 9 months ago

2ynn commented 1 year ago

Fixes #265

Type of change:

[ ] Bug fix (change which fixes an issue).
[x] New feature (change which adds functionality).
[ ] Changes to models (requires making migrations).
[ ] Documentation change.

What's Changed:
1. Set up logging:
1 file for django logs (WARNING level)
1 'saskatoon' file for custom logs (INFO level)
console set to DEBUG level

Added logs when sending emails:
- a log entry is added for each email sent, whether it was successful or not
- refactored notified_unselected_pickers
Removed owner info from "unselected pickers" notification email
Fix href crash when equipment is not associated with any property

Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[ ] New and existing unit tests pass locally with my changes.

2ynn commented 10 months ago

Will deploy this branch on dev server asap to make sure all works as expected before merging.