Merged in fix/BOOK-704-tracking-campaign (pull request #3400)
fix(BOOK-704): add tag name to campaign banner tracking * fix(BOOK-704): add tag name to campaign banner tracking * fix(BOOK-704): handleclose tag name Approved-by: Erik Tiekstra Approved-by: Matilda Landström
This commit is contained in:
@@ -86,7 +86,7 @@ export default function CampaignBanner() {
|
|||||||
}
|
}
|
||||||
|
|
||||||
function handleClose() {
|
function handleClose() {
|
||||||
trackClick("BW close")
|
trackClick(`${campaignBanner?.tag} close`)
|
||||||
setClosedPaths((prev) => new Set(prev).add(pathname))
|
setClosedPaths((prev) => new Set(prev).add(pathname))
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -99,7 +99,10 @@ export default function CampaignBanner() {
|
|||||||
}}
|
}}
|
||||||
>
|
>
|
||||||
<div className={styles.content}>
|
<div className={styles.content}>
|
||||||
<InnerContent link={isOnSamePage ? null : campaignBanner.link}>
|
<InnerContent
|
||||||
|
link={isOnSamePage ? null : campaignBanner.link}
|
||||||
|
tag={campaignBanner.tag}
|
||||||
|
>
|
||||||
{isMobile ? (
|
{isMobile ? (
|
||||||
<MobileCampaignBanner
|
<MobileCampaignBanner
|
||||||
tag={campaignBanner.tag}
|
tag={campaignBanner.tag}
|
||||||
@@ -134,14 +137,16 @@ export default function CampaignBanner() {
|
|||||||
function InnerContent({
|
function InnerContent({
|
||||||
link,
|
link,
|
||||||
children,
|
children,
|
||||||
|
tag,
|
||||||
}: React.PropsWithChildren<{
|
}: React.PropsWithChildren<{
|
||||||
link: CampaignBannerProps["link"]
|
link: CampaignBannerProps["link"]
|
||||||
|
tag: string
|
||||||
}>) {
|
}>) {
|
||||||
return link ? (
|
return link ? (
|
||||||
<NextLink
|
<NextLink
|
||||||
href={link.url}
|
href={link.url}
|
||||||
className={styles.innerContent}
|
className={styles.innerContent}
|
||||||
onClick={() => trackClick("BW campaign banner")}
|
onClick={() => trackClick(`${tag} campaign banner`)}
|
||||||
>
|
>
|
||||||
{children}
|
{children}
|
||||||
</NextLink>
|
</NextLink>
|
||||||
|
|||||||
Reference in New Issue
Block a user