Skip to content

Conversation

@Slashgear
Copy link
Contributor

Why

We have some missing events in Amplitude but we receive them in Segment.
It does seems to be due to the mapping of consent categories with Amplitude.

How

@changeset-bot
Copy link

changeset-bot bot commented Oct 9, 2023

🦋 Changeset detected

Latest commit: e9335c3

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@scaleway/cookie-consent Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@codecov
Copy link

codecov bot commented Oct 9, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (a60a7b2) 98.86% compared to head (e9335c3) 98.86%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1560   +/-   ##
=======================================
  Coverage   98.86%   98.86%           
=======================================
  Files          30       30           
  Lines         792      792           
  Branches      113      113           
=======================================
  Hits          783      783           
  Misses          1        1           
  Partials        8        8           
Files Coverage Δ
...rc/CookieConsentProvider/useSegmentIntegrations.ts 100.00% <100.00%> (ø)

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

@Slashgear Slashgear enabled auto-merge (squash) October 9, 2023 12:02
@philibea philibea disabled auto-merge October 9, 2023 12:02
@philibea philibea merged commit 90d0e18 into main Oct 9, 2023
@philibea philibea deleted the fix-cookie-consents branch October 9, 2023 12:02
@github-actions github-actions bot mentioned this pull request Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants