Skip to content
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

LaunchDarkly: add enable_batching field to Audiences destination action. #1559

Merged
merged 1 commit into from
Sep 12, 2023

Conversation

ldhenry
Copy link
Contributor

@ldhenry ldhenry commented Sep 6, 2023

This PR adds the enable_batching field to LaunchDarkly Audiences' syncAudiences action. This was done to override the default value of false because batching should be enabled in most cases. I decided to make this change after chatting with @joe-ayoub-segment.

Testing

I can see the new field when testing locally.

image

  • Added unit tests for new functionality
  • Tested end-to-end using the local server
  • [Segmenters] Tested in the staging environment

Co-authored-by: Kelly Hofmann <khofmann@launchdarkly.com>
Co-authored-by: Clifford Tawiah <ctawiah@launchdarkly.com>
Co-authored-by: Kelly Hofmann <55991524+k3llymariee@users.noreply.github.com>
Co-authored-by: Cliff Tawiah <82856282+ctawiah@users.noreply.github.com>
Co-authored-by: Molly <molly.jones@launchdarkly.com>
@ldhenry ldhenry requested a review from a team as a code owner September 6, 2023 16:49
@ldhenry ldhenry requested a review from a team September 6, 2023 16:49
@ldhenry ldhenry changed the title Add enable_batching field to LaunchDarkly Audiences destination action. LaunchDarkly: add enable_batching field to Audiences destination action. Sep 6, 2023
@joe-ayoub-segment joe-ayoub-segment self-assigned this Sep 6, 2023
@joe-ayoub-segment joe-ayoub-segment merged commit f332e2e into segmentio:main Sep 12, 2023
@joe-ayoub-segment
Copy link
Contributor

hi @ldhenry this PR has been deployed.

wpride pushed a commit to wpride/action-destinations that referenced this pull request Oct 12, 2023
Co-authored-by: Kelly Hofmann <khofmann@launchdarkly.com>
Co-authored-by: Clifford Tawiah <ctawiah@launchdarkly.com>
Co-authored-by: Kelly Hofmann <55991524+k3llymariee@users.noreply.github.com>
Co-authored-by: Cliff Tawiah <82856282+ctawiah@users.noreply.github.com>
Co-authored-by: Molly <molly.jones@launchdarkly.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants