-
Notifications
You must be signed in to change notification settings - Fork 257
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
Responsys Audiences #2739
base: main
Are you sure you want to change the base?
Responsys Audiences #2739
Conversation
- Corresponding Unit Tests.
New required fields detectedWarning Your PR adds new required fields to an existing destination. Adding new required settings/mappings for a destination already in production requires updating existing customer destination configuration. Ignore this warning if this PR is for a new destination with no active customers in production. The following required fields were added in this PR:
Add these new fields as optional instead and assume default values in |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2739 +/- ##
=======================================
Coverage ? 78.55%
=======================================
Files ? 1040
Lines ? 18982
Branches ? 3603
=======================================
Hits ? 14911
Misses ? 2853
Partials ? 1218 ☔ View full report in Codecov by Sentry. |
This is the same implementation done in #2398, but dedicated to Engage.
Testing