-
Notifications
You must be signed in to change notification settings - Fork 0
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
SD-96364 | Madeira Madeira | Data sent by Segment is not reaching Insider #16
base: main
Are you sure you want to change the base?
Conversation
SD-86748 - Track Event
SD-86750 update user profile action added
SD-86746 - Authentication
|
||
if (userPayload.email_as_identifier) { | ||
// eslint-disable-next-line @typescript-eslint/ban-ts-comment | ||
// @ts-ignore |
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.
It's not a good idea to mute eslinter, but as this bug as urgent, you can proceed
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.
yes it is, but most mutes about types, we can not add types manually, because of that it giving this errors.
description: | ||
'When enabled, the action will send a batch request to Insider. Batches can contain up to 1000 records in a request.', | ||
required: true, | ||
default: false |
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.
can it be default true ? for partners
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.
can be, segment says it can be a delay a bit for batch, because of that I think partner can decide it
SD-102706 Add x-platform-name header.
SD-98474 | [Madeira Madeira] Custom Identifier Improvement
A summary of your pull request, including the what change you're making and why.
Testing
Include any additional information about the testing you have completed to
ensure your changes behave as expected. For a speedy review, please check
any of the tasks you completed below during your testing.