Skip to content

[MAIN] [STRATCONN] Added validation on external_id of character 255 - #3777

Merged
AnkitSegment merged 6 commits into
mainfrom
STRATCONN-6752-klaviyo-actions
May 12, 2026
Merged

[MAIN] [STRATCONN] Added validation on external_id of character 255#3777
AnkitSegment merged 6 commits into
mainfrom
STRATCONN-6752-klaviyo-actions

Address PR review: move external_id check before phone mutation and u…

ca65e65
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded May 12, 2026 in 0s

80.95% (target 70.00%)

View this Pull Request on Codecov

80.95% (target 70.00%)

Details

Codecov Report

❌ Patch coverage is 67.85714% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.95%. Comparing base (139ddfe) to head (ca65e65).

Files with missing lines Patch % Lines
...tion-actions/src/destinations/klaviyo/functions.ts 52.63% 6 Missing and 3 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3777      +/-   ##
==========================================
+ Coverage   80.84%   80.95%   +0.10%     
==========================================
  Files        1377     1351      -26     
  Lines       26417    25325    -1092     
  Branches     5581     5277     -304     
==========================================
- Hits        21357    20501     -856     
+ Misses       4097     3864     -233     
+ Partials      963      960       -3     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.