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

[Klaviyo] Upgrade to revision 2025-01-15 #2748

Merged
merged 1 commit into from
Feb 20, 2025
Merged

[Klaviyo] Upgrade to revision 2025-01-15 #2748

merged 1 commit into from
Feb 20, 2025

Conversation

maryamsharif
Copy link
Contributor

Background: https://docs.google.com/document/d/1B4ixiA_zusdP-VXeBDotQy8GLTjIB2RviMrRvyYAUcg/edit?tab=t.0

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.

  • Added unit tests for new functionality
  • Tested end-to-end using the local server
  • [If destination is already live] Tested for backward compatibility of destination. Note: New required fields are a breaking change.
  • [Segmenters] Tested in the staging environment
  • [Segmenters] [If applicable for this change] Tested for regression with Hadron.

Copy link

codecov bot commented Feb 18, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.50%. Comparing base (e9f28fd) to head (4722845).
Report is 5 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2748   +/-   ##
=======================================
  Coverage   78.50%   78.50%           
=======================================
  Files        1036     1036           
  Lines       18800    18800           
  Branches     3575     3575           
=======================================
  Hits        14759    14759           
  Misses       2836     2836           
  Partials     1205     1205           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@maryamsharif maryamsharif marked this pull request as ready for review February 20, 2025 17:38
@maryamsharif maryamsharif requested a review from a team as a code owner February 20, 2025 17:38
@@ -1,5 +1,5 @@
export const API_URL = 'https://a.klaviyo.com/api'
export const REVISION_DATE = '2023-09-15'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Connecting the dots. This is being used here

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@nick-Ag nick-Ag merged commit dc99299 into main Feb 20, 2025
14 checks passed
@nick-Ag nick-Ag deleted the klaviyo-api-upgrade branch February 20, 2025 18:29
AnkitSegment added a commit that referenced this pull request Feb 27, 2025
commit 71fe28f
Author: Ankit Gupta <[email protected]>
Date:   Tue Feb 25 22:37:15 2025 +0530

    Implemented smart hashing utilities

commit 3116241
Author: harsh-joshi99 <[email protected]>
Date:   Mon Feb 24 13:10:52 2025 +0530

    [Smart Hashing] Add New Smart Hashing Util (#2756)

    * Add new smart hashing util

    * - Add functionality to bypass feature flag check
    - Remove export from smart hashing class
    - Update unit test cases

    * - Add test case with different leaning function
    - Export Features from actions-core

commit 4d90916
Author: Nick Aguilar <[email protected]>
Date:   Fri Feb 21 14:45:48 2025 -0800

    Simplifies type generation for hooks (#2723)

    * WIP - flatten structure of generated hook input and output types

    * Generates new types for LinkedIn & fixes build errors related to those new types

commit 6121cbc
Author: Arijit Ray <[email protected]>
Date:   Fri Feb 21 17:06:52 2025 +0530

    Update Copyright to 2025 (#2754)

commit 19849c6
Author: Sayan Das <[email protected]>
Date:   Fri Feb 21 16:51:56 2025 +0530

    Fix size limit issue on Github Actions (#2757)

commit 8173184
Author: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Date:   Thu Feb 20 10:48:48 2025 -0800

    Publish (#2753)

    Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

commit dc99299
Author: maryamsharif <[email protected]>
Date:   Thu Feb 20 10:29:43 2025 -0800

    Upgrade all actions to the revision 2025-01-15 (#2748)

commit 64d3e5c
Author: Nick Aguilar <[email protected]>
Date:   Thu Feb 20 09:51:07 2025 -0800

    Advanced logging of payloads from Salesforce based on a feature flag (#2749)

commit 1d956c7
Author: Innovative-GauravKochar <[email protected]>
Date:   Thu Feb 20 14:13:06 2025 +0530

    [STRATCONN-5516] | Revert "Worked on fixing braze_id default mappings of  Braze Actions." (#2750)

    * Revert "Worked on Braze Actions identify mapping defaults to reference proper…"

    This reverts commit b0d012e.

    * event type

    ---------

    Co-authored-by: Gaurav Kochar <[email protected]>

commit a41004c
Author: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Date:   Wed Feb 19 14:25:23 2025 -0800

    Publish (#2751)

    Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

commit bcc8bde
Author: rvadera12 <[email protected]>
Date:   Wed Feb 19 14:08:47 2025 -0800

    Adding oauth2.0 to First Party DV360 (#2733)

    * adding oauth to fpdv360

    * remove test files to test locally

    * update unit tests

    * rename test folder

    * remove presets for debugging

    * test

    * add default values path

commit e9f28fd
Author: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Date:   Tue Feb 18 16:52:31 2025 +0530

    Publish (#2747)

    Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

commit 3558814
Author: Sayan Das <[email protected]>
Date:   Tue Feb 18 16:24:59 2025 +0530

    Fix type and linting issues (#2746)

commit 03b2d80
Author: Drew Thompson <[email protected]>
Date:   Tue Feb 18 02:12:31 2025 -0800

    [LAB-1682] Create presets for each standard audience action destination (#2714)

    Co-authored-by: Marín Alcaraz <[email protected]>
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.

3 participants