LesFruitsDefendus / saskatoon-ng

The new generation Saskatoon harverst management system
GNU Affero General Public License v3.0
10 stars 24 forks source link

Better handle failures when sending registration email in member admin #404

Closed 2ynn closed 3 months ago

2ynn commented 4 months ago

Atm sakatoon.log must be checked to investigate failures during send_invite in Onboarding admin action. A better behavior would be to:

  1. move persons whose email was not properly sent to a separate onboarding group with invite_sent = False. All the persons who have successfully received their email can stay in the original batch and be marked the invite_sent = True. This should at least prevent resending email multiple times to the same persons.
  2. dump logs into a comment field so it can be checked directly from the admin panel and ease troubleshooting