Skip to content

Commit

Permalink
Taboola device id fix (#2170)
Browse files Browse the repository at this point in the history
* updating device id field

* fix tests
  • Loading branch information
joe-ayoub-segment authored Jul 16, 2024
1 parent 2009c64 commit dd7c41c
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 17 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,7 @@ describe('Taboola.syncAudience', () => {
event: 'Audience Entered',
type: 'track',
properties: {
ios: {
id: DEVICE_ID
}
ios_id: DEVICE_ID
},
context: {
device: {
Expand Down Expand Up @@ -131,9 +129,7 @@ describe('Taboola.syncAudience', () => {
event: 'Audience Entered',
type: 'track',
properties: {
ios: {
id: DEVICE_ID
}
ios_id: DEVICE_ID
},
context: {
device: {
Expand Down Expand Up @@ -172,9 +168,7 @@ describe('Taboola.syncAudience', () => {
event: 'Audience Entered',
type: 'track',
properties: {
ios: {
id: DEVICE_ID
}
ios_id: DEVICE_ID
},
context: {
device: {
Expand All @@ -196,9 +190,7 @@ describe('Taboola.syncAudience', () => {
event: 'Audience Exited',
type: 'track',
properties: {
ios: {
id: '456'
}
ios_id: '456'
},
context: {
device: {
Expand Down

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Original file line number Diff line number Diff line change
Expand Up @@ -80,15 +80,15 @@ const action: ActionDefinition<Settings, Payload, AudienceSettings> = {
device_id: {
label: 'Mobile Device ID',
description:
"To send iOS and Android Device IDs, include the 'ios.id' and 'android.id' Identifiers from the 'Customized Setup' option when connecting your Audience.",
"To send iOS and Android Device IDs include them as 'ios_id' and 'android_id' via the 'Customized Setup' option when connecting your Audience.",
type: 'string',
required: false,
unsafe_hidden: false,
default: {
'@if': {
exists: { '@path': '$.properties.ios.id' },
then: { '@path': '$.properties.ios.id' },
else: { '@path': '$.properties.android.id' }
exists: { '@path': '$.properties.ios_id' },
then: { '@path': '$.properties.ios_id' },
else: { '@path': '$.properties.android_id' }
}
}
},
Expand Down

0 comments on commit dd7c41c

Please sign in to comment.