From 0974a7471b6f3ae8b015e39ce455ef7351b21d3f Mon Sep 17 00:00:00 2001
From: Max Erenberg <merenber@csclub.uwaterloo.ca>
Date: Sun, 22 Aug 2021 06:06:11 +0000
Subject: [PATCH] ignore UserAlreadySubscribedError

---
 ceod/transactions/members/AddMemberTransaction.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/ceod/transactions/members/AddMemberTransaction.py b/ceod/transactions/members/AddMemberTransaction.py
index dc9b1fc..2479864 100644
--- a/ceod/transactions/members/AddMemberTransaction.py
+++ b/ceod/transactions/members/AddMemberTransaction.py
@@ -4,6 +4,7 @@ from typing import Union, List
 from zope import component
 
 from ..AbstractTransaction import AbstractTransaction
+from ceo_common.errors import UserAlreadySubscribedError
 from ceo_common.interfaces import IConfig, IMailService
 from ceo_common.logger_factory import logger_factory
 from ceod.model import User, Group
@@ -92,6 +93,8 @@ class AddMemberTransaction(AbstractTransaction):
         try:
             user.subscribe_to_mailing_list(self.new_member_list)
             yield 'subscribe_to_mailing_list'
+        except UserAlreadySubscribedError:
+            pass
         except Exception as err:
             logger.warning('subscribe_to_mailing_list failed:\n' + traceback.format_exc())
             yield 'failed_to_subscribe_to_mailing_list: ' + str(err)