Currently one can only set language during subscription. However, one might want to provide additional languages at a later time to users and/or allow them to switch their language preference when chosen incorrectly.

Therefore it would be helpful if the language argument could be passend along.

CommentFileSizeAuthor
#7 3091584-7-member-update-language-2.x.patch2.58 KBmilanbombschliip
#6 3091584-6-member-update-language-2.x.patch2.82 KBtcrawford
#2 3091584-2-member-update-language.patch2.81 KBgrahl

Issue fork mailchimp-3091584

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

grahl created an issue. See original summary.

grahl’s picture

grahl
he/him
Location Zürich
CreditAttribution: grahl at Liip commented
FileSize
3091584-2-member-update-language.patch2.81 KB

Tried to keep the parameter order in sync with mailchimp_subscribe() instead of appending it after gdpr_consent.

grahl’s picture

grahl
he/him
Location Zürich
CreditAttribution: grahl at Liip commented
Status: Active » Needs review
tcrawford’s picture

tcrawford CreditAttribution: tcrawford at Liip commented
Version: 8.x-1.x-dev » 2.x-dev

tcrawford opened merge request !22

tcrawford’s picture

tcrawford CreditAttribution: tcrawford at Liip commented
FileSize
3091584-6-member-update-language-2.x.patch2.82 KB

I have created an issue fork, MR and patch for the 2.x branch.

milanbombschliip’s picture

milanbombschliip CreditAttribution: milanbombschliip commented
FileSize
3091584-7-member-update-language-2.x.patch2.58 KB

Rerolled for version drupal/mailchimp 2.x

Status: Needs review » Needs work

The last submitted patch, 7: 3091584-7-member-update-language-2.x.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.