Skip to content

Commit 1ed0738

Browse files
authored
Merge pull request #4749 from mozilla/mntor-3337
Add telemetry to PetitionBanner (MNTOR-3337)
2 parents 45da49c + 1ab61dc commit 1ed0738

File tree

1 file changed

+30
-6
lines changed

1 file changed

+30
-6
lines changed

src/app/components/client/PetitionBanner.tsx

+30-6
Original file line numberDiff line numberDiff line change
@@ -9,14 +9,16 @@ import { useL10n } from "../../hooks/l10n";
99
import { useLocalDismissal } from "../../hooks/useLocalDismissal";
1010
import { useHasRenderedClientSide } from "../../hooks/useHasRenderedClientSide";
1111
import styles from "./PetitionBanner.module.scss";
12-
import { Button } from "./Button";
1312
import { CONST_URL_DATA_PRIVACY_PETITION_BANNER } from "../../../constants";
13+
import { TelemetryButton } from "./TelemetryButton";
14+
import { useTelemetry } from "../../hooks/useTelemetry";
1415

1516
export const PetitionBanner = () => {
1617
const l10n = useL10n();
1718

1819
const hasRenderedClientSide = useHasRenderedClientSide();
1920
const localDismissal = useLocalDismissal("data_privacy_petition_banner");
21+
const recordTelemetry = useTelemetry();
2022

2123
if (!hasRenderedClientSide || localDismissal.isDismissed) {
2224
return null;
@@ -35,24 +37,46 @@ export const PetitionBanner = () => {
3537
})}
3638
</p>
3739
<div className={styles.buttons}>
38-
<Button
40+
<TelemetryButton
3941
variant="primary"
4042
className={styles.signButton}
4143
href={CONST_URL_DATA_PRIVACY_PETITION_BANNER}
4244
target="_blank"
45+
event={{
46+
module: "ctaButton",
47+
name: "click",
48+
data: {
49+
button_id: "sign_petition",
50+
},
51+
}}
4352
>
4453
{l10n.getString("petition-banner-data-privacy-button-sign")}
45-
</Button>
46-
<Button
54+
</TelemetryButton>
55+
<TelemetryButton
4756
variant="tertiary"
4857
className={styles.dismissButton}
4958
onPress={() => dismiss()}
59+
event={{
60+
module: "button",
61+
name: "click",
62+
data: {
63+
button_id: "petition_no_thank_you",
64+
},
65+
}}
5066
>
5167
{l10n.getString("petition-banner-data-privacy-button-dismiss")}
52-
</Button>
68+
</TelemetryButton>
5369
</div>
5470
</div>
55-
<button className={styles.closeButton} onClick={() => dismiss()}>
71+
<button
72+
className={styles.closeButton}
73+
onClick={() => {
74+
dismiss();
75+
recordTelemetry("button", "click", {
76+
button_id: "petition_dismiss",
77+
});
78+
}}
79+
>
5680
<CloseBtn
5781
alt={l10n.getString("survey-csat-survey-dismiss-label")}
5882
width="14"

0 commit comments

Comments
 (0)