- Issue created by @krystalcode
- @krystalcode opened merge request.
- Status changed to Needs work
about 1 year ago 3:08pm 27 November 2023 - π³π±Netherlands megachriz
I see that the changes make sense.
mailchimp_cron()
will callcall_user_func_array()
with named arguments and one of the named arguments -$tags
- is not defined currently inmailchimp_update_member_process()
.I do think we need to do a little bit more.
mailchimp_update_member_process()
doesn't do anything with the$tags
parameter now. So I think that the code should make a call toaddTagsMember()
when$tags
is not empty. Also,$tags
probably need to passed whenmailchimp_update_member_process()
is called in the same function again (Β± line 707):return mailchimp_update_member_process($list_id, $email, $merge_vars, $interests, $format, TRUE, $gdpr_consent);
- Status changed to Closed: duplicate
about 1 year ago 9:59pm 15 December 2023 - πΊπΈUnited States xenophyle
Duplicate of π Update mailchimp_update_member_process for $tags Active
- π§π·Brazil brandonlira
brandonlira β changed the visibility of the branch 3352943-unknown-named-parameter to hidden.
- π§π·Brazil brandonlira
brandonlira β changed the visibility of the branch 3352943-unknown-named-parameter to active.