From 43b1d9ceabac8e8843509e3713675273666ed6a6 Mon Sep 17 00:00:00 2001 From: Robert Helmer Date: Fri, 20 Dec 2024 11:50:50 -0800 Subject: [PATCH] Remove TODOs per flozia --- src/app/hooks/useGlean.ts | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/app/hooks/useGlean.ts b/src/app/hooks/useGlean.ts index bf5e57c8b5..d7fe185b95 100644 --- a/src/app/hooks/useGlean.ts +++ b/src/app/hooks/useGlean.ts @@ -36,8 +36,6 @@ export const useGlean = () => { // Record the `plan_tier` key on all events. // `plan_tier` is set on every metric, but it's too much work for TypeScript // to infer that — hence the type assertion. - // - // TODO can we fix this? It looks like these are only for button->click which is misleading. (data as GleanMetricMap["button"]["click"]).plan_tier = isPremiumUser ? "Plus" : "Free"; @@ -45,8 +43,6 @@ export const useGlean = () => { // Record the `nimbus_*` keys on all events. // `nimbus_*` is set on every metric, but it's too much work for TypeScript // to infer that — hence the type assertion. - // - // TODO can we fix this? It looks like these are only for button->click which is misleading. console.debug({ event, experimentData }); if (experimentData) { (data as GleanMetricMap["button"]["click"]).nimbus_user_id =