-
Notifications
You must be signed in to change notification settings - Fork 240
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 as PETs mapping #2398
base: main
Are you sure you want to change the base?
Conversation
- Implementing Send Audience as PET action.
- Improving Send Audiences as PET payload to work with the new parameters.
hi @seg-leonelsanches - thanks for raising this PR. Looks like this is for a new Action. Could you complete the form on this page please so that we can properly evaluate if a new Action is the right way to go please? I'm not a Responsys SME so I'd like to loop in @mayur-pitale and @longstoryshort as they were involved with writing the Integration. |
Hi @joe-ayoub-segment. You're welcome. Here's the supporting document: https://segment.atlassian.net/wiki/spaces/~62b2d761d337d0b7d09d6455/pages/2874376194/Responsys+PET+per+Audience+review+form |
Actions Responsys today tries to work with existing PETs. The problem with this is: there’s no way to automatically modify a PET to include new columns when a corresponding audience is created. The actual endpoints only allow us to create new PETs.
The idea is a new Action that creates a new PET for each new audience before sending records to Responsys, also validating if users are already part of the same Profile List the PET belongs. This approach is already used with our customers successfully, but implemented as a Destination Function.
This implementation also follows the cadence that Responsys expects, with one event at a time, waiting for each API response, instead of parallel Promises.
Testing
Payload suggestion for
/refreshAccessToken
:Payload suggestion for
/sendAudienceAsPet
: