From 3da61989b7c038a39a37c4f195749fc3ba71b686 Mon Sep 17 00:00:00 2001 From: Francis Cao Date: Sun, 13 Apr 2025 18:31:59 -0700 Subject: [PATCH] fix sorting on UTM queries, add more ads in case logic --- src/queries/sql/reports/getAttribution.ts | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/src/queries/sql/reports/getAttribution.ts b/src/queries/sql/reports/getAttribution.ts index 62003222a..62ad068a5 100644 --- a/src/queries/sql/reports/getAttribution.ts +++ b/src/queries/sql/reports/getAttribution.ts @@ -61,7 +61,7 @@ async function relationalQuery( and we.session_id = m.session_id ${currency ? '' : `where we.${utmColumn} != ''`} group by 1 - order by name desc + order by 2 desc limit 20`; } @@ -269,7 +269,7 @@ async function clickhouseQuery( and we.session_id = m.session_id ${currency ? '' : `where we.${utmColumn} != ''`} group by 1 - order by name desc + order by 2 desc limit 20`; } @@ -361,7 +361,12 @@ async function clickhouseQuery( ` ${currency ? revenueEventQuery : eventQuery} ${getModelQuery(model)} - select multiIf(gclid != '', 'Google', fbclid != '', 'Facebook', '') name, + select multiIf(gclid != '', 'Google Ads', + fbclid != '', 'Facebook / Meta', + msclkid != '', 'Microsoft Ads', + ttclid != '', 'TikTok Ads', + li_fat_id != '', ' LinkedIn Ads', + twclid != '', 'Twitter Ads (X)','') name, ${currency ? 'sum(e.value)' : 'uniqExact(we.session_id)'} value from events e join model m