- Issue created by @durifal
- @durifal opened merge request.
- Issue was unassigned.
- Status changed to Needs review
about 1 year ago 12:35pm 26 September 2023 - πΈπ°Slovakia durifal
Here is also .patch file with the fix, if anybody would need it.
- Status changed to Needs work
about 1 year ago 1:44pm 26 September 2023 - π¦πΉAustria klausi π¦πΉ Vienna
Thanks, approach makes sense to me!
Found just some minor comment issues.
-
+++ b/includes/sendinblue.manage.inc (date 1695300377393) @@ -476,4 +476,34 @@ + * Subscriber user.
Wrong comment, should be unsubscribe?
-
+++ b/includes/sendinblue.manage.inc (date 1695300377393) @@ -476,4 +476,34 @@ + * @param string $listid + * An array of list id.
but the data type is string. should it be an array or a string?
-