public
/
pyceo
27
1
Fork 3
Code Issues 25 Pull Requests 5 Projects Releases 1 Wiki Activity

add failed_to_subscribe_to_mailing_lists
continuous-integration/drone/pr Build is passing Details

This commit is contained in:
Max Erenberg 2021-08-22 06:07:56 +00:00
parent f497612399
commit ca3ada5886
1 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
ceod/transactions/members/UpdateMemberPositionsTransaction.py
View File

@ -78,6 +78,7 @@ class UpdateMemberPositionsTransaction(AbstractTransaction):
yield 'update_exec_group_ldap'
# Update mailing list subscriptions
subscription_failed = False
for username, new_positions in self.positions.items():
user = self.users[username]
for mailing_list in mailing_lists:
@ -90,7 +91,11 @@ class UpdateMemberPositionsTransaction(AbstractTransaction):
pass
except Exception:
logger.warning(f'Failed to update mailing list for {user.uid}')
yield 'subscribe_to_mailing_lists'
subscription_failed = True
if subscription_failed:
yield 'failed_to_subscribe_to_mailing_lists'
else:
yield 'subscribe_to_mailing_lists'
self.finish(new_positions_reversed)