👌 [#2192] PR feedback

maykinmedia · Mar 15, 2024 · 86d3e57 · 86d3e57
1 parent 3a44a8c
commit 86d3e57
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/src/open_inwoner/accounts/views/profile.py b/src/open_inwoner/accounts/views/profile.py
@@ -332,6 +332,9 @@ def form_valid(self, form):
 
         # Display errors raised by Laposta API
         if form.errors:
+            self.log_user_action(
+                self.request.user, _("failed to modify user newsletter subscription")
+            )
             return self.form_invalid(form)
 
         messages.success(self.request, _("Uw wijzigingen zijn opgeslagen"))

diff --git a/src/open_inwoner/laposta/client.py b/src/open_inwoner/laposta/client.py
@@ -63,6 +63,7 @@ def remove_subscription(self, list_id: str, member_id: str) -> Member | None:
             # Handle scenario where a subscription does not exists in the API,
             # but it does exist locally
             if error.get("code") == 203 and error.get("parameter") == "member_id":
+                logger.info("Subscription does not exist for user")
                 return None
 
         data = get_json_response(response)