Hello,

There is no config schema for the mailchimp lists field.

I will upload a patch.

CommentFileSizeAuthor
#5 interdiff-mailchimp-config_schema-2883710-3-4.txt511 bytesrjacobsen0
#5 mailchimp-config_schema-2883710-4.patch2.31 KBrjacobsen0
#3 mailchimp-config_schema-2883710-3.patch2.31 KBjacktonkin
#3 interdiff.txt1.93 KBjacktonkin
#2 mailchimp-config_schema-2883710-2.patch2.51 KBgrimreaper

Comments

Grimreaper created an issue. See original summary.

grimreaper’s picture

grimreaper
Primary language French
Location France 🇫🇷
commented
Assigned: grimreaper » Unassigned
Status: Active » Needs review
StatusFileSize
new mailchimp-config_schema-2883710-2.patch2.51 KB

Here is the patch.

Thanks for the review.

jacktonkin’s picture

jacktonkin commented
StatusFileSize
new interdiff.txt1.93 KB
new mailchimp-config_schema-2883710-3.patch2.31 KB

Updating for field settings added since the last patch. In particular this allows the subscribe checkbox label to be translated.

Also updated some labels to match the field forms.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Reviewed & tested by the community

Would be nice to get this in.

rjacobsen0’s picture

rjacobsen0 commented
StatusFileSize
new mailchimp-config_schema-2883710-4.patch2.31 KB
new interdiff-mailchimp-config_schema-2883710-3-4.txt511 bytes

We made an effort to match Mailchimp's brand by using a lowercase 'c' in Mailchimp. Let's keep that going. Also Lists are now known as Audiences for all user-facing strings.

  • rjacobsen0 committed 20c22fd on 8.x-1.x 
    Issue #2883710 by jacktonkin, rjacobsen0, Grimreaper: Add mailchimp...
rjacobsen0’s picture

rjacobsen0 commented

The reason the last patch did not apply is because I committed before running the test.

rjacobsen0’s picture

rjacobsen0 commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.