-
Notifications
You must be signed in to change notification settings - Fork 107
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
feat: update klaviyo api version (#1981)
* feat: update klaviyo api version * fix: correct batching logic * feat: update Klaviyo API version and add batching for subscribe endpoint * feat: update group call and test cases * feat: update batching logic for group call * feat: make consent field backward-compatible * fix: consent conditional check * feat: add consent in mapping and update mapping * fix: remove extra comments and add optional chaining * feat: add total and value mapping for value and remove extra lines * feat: add conditon for failed request * feat: add test coverage * feat: refactor code and improve coverage * feat: add optional chaining * fix: make use of common variable * fix: profile id parsing * chore: add negative router test cases * fix: lint errors * fix: correct event extraction * chore: refactor code * fix(klaviyo): add array index for subsribe reponse array in batching logic * chore: refactor batching logic, and break the functions and move them to util * chore: refactor batching logic, and break the functions and move them to util * chore: refactor code * chore: rename function parameters to relevant names * chore: refactor code * chore: refactor code * chore: update parameter names add documentation for function * fix: sonar code-smell & change forEach to map Signed-off-by: Sai Sankeerth <sanpj2292@github.com> * fix(klaviyo): fix bugs created due to last few refactor commits * fix(klaviyo): add _id to customProperties * fix(klaviyo): remove unused import --------- Signed-off-by: Sai Sankeerth <sanpj2292@github.com> Co-authored-by: Sai Sankeerth <sanpj2292@github.com>
- Loading branch information
Showing
11 changed files
with
1,119 additions
and
1,149 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,85 @@ | ||
| [ | ||
| { | ||
| "destKey": "$email", | ||
| "sourceKeys": "email", | ||
| "required": false, | ||
| "sourceFromGenericMap": true | ||
| }, | ||
| { | ||
| "destKey": "$first_name", | ||
| "sourceKeys": "firstName", | ||
| "required": false, | ||
| "sourceFromGenericMap": true | ||
| }, | ||
| { | ||
| "destKey": "$last_name", | ||
| "sourceKeys": "lastName", | ||
| "required": false, | ||
| "sourceFromGenericMap": true | ||
| }, | ||
| { | ||
| "destKey": "$phone_number", | ||
| "sourceKeys": "phone", | ||
| "required": false, | ||
| "sourceFromGenericMap": true | ||
| }, | ||
| { | ||
| "destKey": "$city", | ||
| "sourceKeys": [ | ||
| "traits.city", | ||
| "traits.address.city", | ||
| "context.traits.city", | ||
| "context.traits.address.city", | ||
| "properties.city" | ||
| ], | ||
| "required": false | ||
| }, | ||
| { | ||
| "destKey": "$region", | ||
| "sourceKeys": [ | ||
| "traits.region", | ||
| "traits.address.region", | ||
| "context.traits.region", | ||
| "context.traits.address.region", | ||
| "properties.region" | ||
| ], | ||
| "required": false | ||
| }, | ||
| { | ||
| "destKey": "$country", | ||
| "sourceKeys": [ | ||
| "traits.country", | ||
| "traits.address.country", | ||
| "context.traits.country", | ||
| "context.traits.address.country", | ||
| "properties.country" | ||
| ], | ||
| "required": false | ||
| }, | ||
| { | ||
| "destKey": "$zip", | ||
| "sourceKeys": [ | ||
| "traits.zip", | ||
| "traits.postalcode", | ||
| "traits.address.zip", | ||
| "traits.address.postalcode", | ||
| "context.traits.zip", | ||
| "context.traits.postalcode", | ||
| "context.traits.address.zip", | ||
| "context.traits.address.postalcode", | ||
| "properties.zip", | ||
| "properties.postalcode" | ||
| ], | ||
| "required": false | ||
| }, | ||
| { | ||
| "destKey": "$consent", | ||
| "sourceKeys": ["context.traits.consent", "traits.consent", "properties.consent"], | ||
| "required": false | ||
| }, | ||
| { | ||
| "destKey": "$image", | ||
| "sourceKeys": ["traits.image", "context.traits.image", "properties.image"], | ||
| "required": false | ||
| } | ||
| ] |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,14 +1,14 @@ | ||
| [ | ||
| { | ||
| "destKey": "event", | ||
| "destKey": "metric.name", | ||
| "sourceKeys": ["event", "properties.event", "properties.name"], | ||
| "required": true | ||
| }, | ||
| { | ||
| "destKey": "properties", | ||
| "sourceKeys": "properties", | ||
| "metadata": { | ||
| "excludes": ["email", "phone"] | ||
| "excludes": ["email", "phone", "revenue", "total", "value"] | ||
| } | ||
| } | ||
| ] |
Oops, something went wrong.