From 5d67f52c3b844821ad5191a1d840a0d1e464ee12 Mon Sep 17 00:00:00 2001 From: Yohann Fabri <11197007+YoranSys@users.noreply.github.com> Date: Fri, 8 Sep 2023 17:35:30 +0200 Subject: [PATCH] fix(#54): SubscriberApi.list() not returning credentials, field email is missing in SubscriberDto (#55) --- novu/dto/subscriber.py | 2 +- tests/api/test_subscriber.py | 2 -- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/novu/dto/subscriber.py b/novu/dto/subscriber.py index 9004d808..35af0b9c 100644 --- a/novu/dto/subscriber.py +++ b/novu/dto/subscriber.py @@ -80,7 +80,7 @@ class SubscriberDto(CamelCaseDto["SubscriberDto"]): # pylint: disable=R0902 subscriber_id: str """Subscriber's ID""" - email: str + email: Optional[str] = None """Email of the subscriber""" first_name: Optional[str] = None diff --git a/tests/api/test_subscriber.py b/tests/api/test_subscriber.py index a8001b2d..9f5042be 100644 --- a/tests/api/test_subscriber.py +++ b/tests/api/test_subscriber.py @@ -31,14 +31,12 @@ def setUpClass(cls) -> None: "updatedAt": "2023-02-06T23:03:22.645Z", "__v": 0, "isOnline": False, - "email": "oscar.marie-taillefer@spikeelabs.fr", "lastOnlineAt": "2023-02-06T23:03:22.645Z", } cls.response_list = {"page": 0, "totalCount": 1, "pageSize": 10, "data": [cls.subscriber_json]} cls.response_get = {"data": cls.subscriber_json} cls.expected_dto = SubscriberDto( subscriber_id="63dafed4117f8c850991ec4a", - email="oscar.marie-taillefer@spikeelabs.fr", first_name=None, last_name=None, phone=None,