Skip to content

fix(storefront): Persist awc as a cookie so it survives a new tab - #789

Merged
leomp12 merged 2 commits into
mainfrom
fix/awin-conversion-tracking
Aug 1, 2026
Merged

fix(storefront): Persist awc as a cookie so it survives a new tab#789
leomp12 merged 2 commits into
mainfrom
fix/awin-conversion-tracking

Conversation

@vitorrgg

Copy link
Copy Markdown
Member

Awin's own "last click direto" test scenario (affiliate click, then
open a fresh tab and navigate directly) was losing the awc value
because it was only ever stored in sessionStorage, which a manually
opened tab does not inherit. The fallback-cookie read logic already
existed but nothing wrote that cookie — now it does.

@vitorrgg
vitorrgg requested a review from leomp12 July 30, 2026 18:05
vitorrgg and others added 2 commits July 31, 2026 21:16
Awin's own "last click direto" test scenario (affiliate click, then
open a fresh tab and navigate directly) was losing the awc value
because it was only ever stored in sessionStorage, which a manually
opened tab does not inherit. The fallback-cookie read logic already
existed but nothing wrote that cookie — now it does.
Awin's Conversion API doc doesn't enumerate accepted "channel" values
(channel is free-form and 'aw' is the mandated fallback), so stop
claiming it does; point the fallback pixel to its actual doc page and
trim change-narration comments down to the constraints they document.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01D6txsobQ3CPAo1yYK7XDab
@leomp12
leomp12 force-pushed the fix/awin-conversion-tracking branch from 6e08245 to 10d22af Compare August 1, 2026 00:16
@leomp12 leomp12 changed the title Persist awc as a cookie so it survives a new tab fix(storefront): Persist awc as a cookie so it survives a new tab Aug 1, 2026

@leomp12 leomp12 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Effective delta vs main is the 7-line awc cookie write, validated against Awin's official Conversion API docs. Comment cleanup applied on the branch.

@leomp12
leomp12 merged commit 1760abc into main Aug 1, 2026
2 checks passed
@leomp12
leomp12 deleted the fix/awin-conversion-tracking branch August 1, 2026 00:19
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.

2 participants