Tim Abbott cde86fbaca send_email: Move logging after dry-run abort. ...

Previously, the output would make it look like we sent an actual email
to the first user in the dry_run output, which is very confusing.

The `dry_run` code path already prints all the accounts that would
have been emailed at the end, so there's no reason to have this line
before the dry_run check.

Additionally, we move after the `get_connection` check because
failures at that stage shouldn't result in logging an attempt to send
an email.

2021-08-05 10:15:49 -07:00

20 KiB

Raw Blame History

View Raw