Skip to content

Conversation

@chambo-e
Copy link
Contributor

If we provided an empty or partial environment we would get &&cookies_win=x in the requested url

@chambo-e chambo-e requested a review from a team April 12, 2022 15:40
@chambo-e chambo-e self-assigned this Apr 12, 2022
@chambo-e chambo-e requested review from DorianMaliszewski and vincentaudebert and removed request for a team April 12, 2022 15:40
@codecov
Copy link

codecov bot commented Apr 12, 2022

Codecov Report

Merging #719 (0d91aa7) into main (ad0dbcc) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main      #719   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           20        20           
  Lines          573       573           
  Branches        86        86           
=========================================
  Hits           573       573           
Impacted Files Coverage Δ
packages/use-gtm/src/scripts.ts 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ad0dbcc...0d91aa7. Read the comment docs.

@chambo-e chambo-e merged commit 6744edb into main Apr 20, 2022
@chambo-e chambo-e deleted the fix/gtm_double branch April 20, 2022 08:23
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.

4 participants