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

fix(mc): handle standard merge fields when fixing duplicates #3637

Open
wants to merge 1 commit into
base: release
Choose a base branch
from

Conversation

dkoo
Copy link
Contributor

@dkoo dkoo commented Dec 18, 2024

All Submissions:

Changes proposed in this Pull Request:

Allows our script to consolidate and fix duplicate merge fields to also handle standard Mailchimp merge fields (First/Last Name, Phone, Address) in addition to Newspack-specific fields.

How to test the changes in this Pull Request:

  1. In your connected Mailchimp audience, create duplicate merge fields for First Name, Last Name, Phone, and Address. These fields usually exist for every Mailchimp audience and can also be a part of our contact metadata payload, which means they could be duplicated by our sync methods.
  2. Fill out these fields for some existing contacts in your audience.
  3. Run wp newspack mailchimp merge-fields fix-duplicates --dry-run and confirm that the duplicate fields are reported.
  4. Run wp newspack mailchimp merge-fields fix-duplicates and confirm that the duplicate fields are deleted in Mailchimp, and that the info you filled out in step 2 get consolidated into the original instances of those fields.

Other information:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes, as applicable?
  • Have you successfully ran tests with your changes locally?

@dkoo dkoo added the [Status] Needs Review The issue or pull request needs to be reviewed label Dec 18, 2024
@dkoo dkoo self-assigned this Dec 18, 2024
@dkoo dkoo requested a review from a team as a code owner December 18, 2024 17:04
@dkoo dkoo marked this pull request as draft December 18, 2024 17:28
@dkoo dkoo marked this pull request as ready for review December 18, 2024 21:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Status] Needs Review The issue or pull request needs to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant