-
Notifications
You must be signed in to change notification settings - Fork 22
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
feat: add methods that were in mailchimp data connector #1568
feat: add methods that were in mailchimp data connector #1568
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## epic/consolidate-data-flows #1568 +/- ##
=================================================================
+ Coverage 17.78% 18.27% +0.49%
- Complexity 2296 2302 +6
=================================================================
Files 44 44
Lines 8706 8731 +25
=================================================================
+ Hits 1548 1596 +48
+ Misses 7158 7135 -23 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Working as expected here too 👍
All Submissions:
Changes proposed in this Pull Request:
Brings the updated functionality of preparing merge fields that was present in the Mailchimp data api connector
How to test the changes in this Pull Request:
See Automattic/newspack-plugin#3237
Other information: