Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Trinidad & Tobago country incorrectly synced #1863

Closed
kamilszewczyk opened this issue Jan 5, 2024 · 2 comments · Fixed by #1886, #1887, #1888 or #1889
Closed

Trinidad & Tobago country incorrectly synced #1863

kamilszewczyk opened this issue Jan 5, 2024 · 2 comments · Fixed by #1886, #1887, #1888 or #1889

Comments

@kamilszewczyk
Copy link

Issue Description

When a customer has an address with a Trinidad & Tobago country selected, when syncing to Mailchimp the contact information shows USA as the country

Preconditions

1. Magento Cloud 2.4.3-p1
2. mc-magento2 103.4.57.
3. ADDRESS added as the Audience field
4. Ecommerce sync enabled

Steps to reproduce

Register as a customer with an address of Trinidad & Tobago country
Wait/trigger mailchimp sync of new customer

Actual and Expected result

Expected result:
Contact in mailchimp added with Trinidad and Tobago country
Actual result:
Contact in mailchimp added with USA country

Additional information

I'm pretty sure it's cause by the & (&) sign in the Magento country name, when mailchimp expects the "and".
I confirmed it, by hardcoding the country in the customer address data sent to mailchimp to "Trinidad and Tobago" and it has created correct address in mailchimp.
I'm not sure if it'll be enough to do a quick replace of ampersands with "and" keyword - this seems to be working correctly for Trinidad, however I have not tested it for any other countries with ampersands in their names.

@gonzaloebiz
Copy link
Collaborator

Hi @kamilszewczyk

try the fix on here 4dfc168
We will include this in the next release

Best

@gonzaloebiz
Copy link
Collaborator

No news in one month, close the issue

gonzaloebiz added a commit that referenced this issue Feb 23, 2024
gonzaloebiz added a commit that referenced this issue Feb 23, 2024
gonzaloebiz added a commit that referenced this issue Feb 23, 2024
gonzaloebiz added a commit that referenced this issue Feb 23, 2024
gonzaloebiz added a commit that referenced this issue Feb 23, 2024
gonzaloebiz added a commit that referenced this issue Feb 23, 2024
gonzaloebiz added a commit that referenced this issue Feb 23, 2024
gonzaloebiz added a commit that referenced this issue Feb 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment