From 3725a3fa1bb72aaf434357062837399672cf7df4 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Alexa=20Valentov=C3=A1?= <git@imaniti.org>
Date: Wed, 26 Mar 2025 15:54:07 +0100
Subject: [PATCH] Fix Ecomail newsletter tag updates
---
shared/utils.py | 20 ++++++++++++++++----
1 file changed, 16 insertions(+), 4 deletions(-)
diff --git a/shared/utils.py b/shared/utils.py
index 657c6105..1adb7eb0 100644
--- a/shared/utils.py
+++ b/shared/utils.py
@@ -72,11 +72,11 @@ def subscribe_to_newsletter(email, list_id):
def subscribe_to_ecomail_newsletter(
email: str,
- categories: str,
+ tags: str,
source: str,
list_id: int | None = None,
custom_data: dict | None = None,
- update_existing: bool = True
+ update_existing: bool = True,
):
if custom_data is None:
custom_data = {}
@@ -84,13 +84,25 @@ def subscribe_to_ecomail_newsletter(
if not list_id:
list_id = settings.ECOMAIL_DEFAULT_LIST_ID
- split_categories = categories.split(",") if categories is not None else []
+ existing_subscriber_data = requests.get(
+ urljoin(settings.ECOMAIL_API_URL, f"lists/{list_id}/subscriber/{email}"),
+ headers={"key": settings.ECOMAIL_API_KEY},
+ )
+
+ existing_tags = []
+
+ if existing_subscriber_data.status_code == 200:
+ existing_subscriber_data = existing_subscriber_data.json()
+ existing_tags = existing_subscriber_data["subscriber"]["tags"]
+
+ split_tags = tags.split(",") if tags is not None else []
+ split_tags += existing_tags
data = {
"subscriber_data": {
"email": email,
"source": source,
- "tags": split_categories,
+ "tags": split_tags,
**custom_data,
},
"update_existing": update_existing,
--
GitLab