Skip to content

STRATCONN-5986 - [Facebook Pixel] - minor updates before Public beta#3615

Open
joe-ayoub-segment wants to merge 5 commits intomainfrom
fb-pixel-update
Open

STRATCONN-5986 - [Facebook Pixel] - minor updates before Public beta#3615
joe-ayoub-segment wants to merge 5 commits intomainfrom
fb-pixel-update

Conversation

@joe-ayoub-segment
Copy link
Contributor

Some minor tweaks after meeting with facebook.

  • adds hashing for PII
  • more unit tests

Testing

Unit tests added. New integration.

Security Review

Please ensure sensitive data is properly protected in your integration.

  • Reviewed all field definitions for sensitive data (API keys, tokens, passwords, client secrets) and confirmed they use type: 'password'

New Destination Checklist

  • Extracted all action API versions to verioning-info.ts file. example

@codecov
Copy link

codecov bot commented Feb 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 31.86%. Comparing base (c5c75a3) to head (d62b494).

❌ Your project check has failed because the head coverage (31.86%) is below the target coverage (70.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #3615       +/-   ##
===========================================
- Coverage   80.34%   31.86%   -48.49%     
===========================================
  Files        1283       15     -1268     
  Lines       23583      747    -22836     
  Branches     4728      125     -4603     
===========================================
- Hits        18947      238    -18709     
+ Misses       3755      509     -3246     
+ Partials      881        0      -881     

☔ 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.

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.

1 participant

Comments