From 22333a1e9a71353d11f01e452791c115a371535b Mon Sep 17 00:00:00 2001 From: Francis Cao Date: Fri, 9 Aug 2024 18:20:26 -0700 Subject: [PATCH] remove event from website stats. doesnt work with view --- .../[websiteId]/events/EventsMetricsBar.tsx | 5 ----- src/queries/analytics/getWebsiteStats.ts | 13 +++---------- .../analytics/sessions/getWebsiteSessions.ts | 8 ++++---- 3 files changed, 7 insertions(+), 19 deletions(-) diff --git a/src/app/(main)/websites/[websiteId]/events/EventsMetricsBar.tsx b/src/app/(main)/websites/[websiteId]/events/EventsMetricsBar.tsx index f77c6c04..0a054e8d 100644 --- a/src/app/(main)/websites/[websiteId]/events/EventsMetricsBar.tsx +++ b/src/app/(main)/websites/[websiteId]/events/EventsMetricsBar.tsx @@ -28,11 +28,6 @@ export function EventsMetricsBar({ websiteId }: { websiteId: string }) { label={formatMessage(labels.views)} formatValue={formatLongNumber} /> - diff --git a/src/queries/analytics/getWebsiteStats.ts b/src/queries/analytics/getWebsiteStats.ts index a33011ff..c35aea06 100644 --- a/src/queries/analytics/getWebsiteStats.ts +++ b/src/queries/analytics/getWebsiteStats.ts @@ -33,15 +33,12 @@ async function relationalQuery( count(distinct t.session_id) as "visitors", count(distinct t.visit_id) as "visits", count(distinct t.country) as "countries", - count(t.event_name) as "countries", - sum(case when t.event_type = 2 then 1 else 0 end) as "events", sum(case when t.c = 1 then 1 else 0 end) as "bounces", sum(${getTimestampDiffSQL('t.min_time', 't.max_time')}) as "totaltime", from ( select website_event.session_id, website_event.visit_id, - website_event.event_type, session.country, count(*) as "c", min(website_event.created_at) as "min_time", @@ -51,7 +48,7 @@ async function relationalQuery( where website_event.website_id = {{websiteId::uuid}} and website_event.created_at between {{startDate}} and {{endDate}} ${filterQuery} - group by 1, 2, 3, 4 + group by 1, 2, 3 ) as t `, params, @@ -78,14 +75,12 @@ async function clickhouseQuery( uniq(t.session_id) as "visitors", uniq(t.visit_id) as "visits", uniq(t.country) as "countries", - sumIf(1, t.event_type = 2) as "events", sumIf(1, t.c = 1) as "bounces", sum(max_time-min_time) as "totaltime" from ( select session_id, visit_id, - event_type, country, count(*) c, min(created_at) min_time, @@ -94,7 +89,7 @@ async function clickhouseQuery( where website_id = {websiteId:UUID} and created_at between {startDate:DateTime64} and {endDate:DateTime64} ${filterQuery} - group by session_id, visit_id, event_type, country + group by session_id, visit_id, country ) as t; `; } else { @@ -104,14 +99,12 @@ async function clickhouseQuery( uniq(session_id) as "visitors", uniq(visit_id) as "visits", uniq(country) as "countries", - sumIf(1, t.event_type = 2) as "events", sumIf(1, t.c = 1) as "bounces", sum(max_time-min_time) as "totaltime" from ( select session_id, visit_id, - event_type, country, sum(views) c, min(min_time) min_time, @@ -120,7 +113,7 @@ async function clickhouseQuery( where website_id = {websiteId:UUID} and created_at between {startDate:DateTime64} and {endDate:DateTime64} ${filterQuery} - group by session_id, visit_id, event_type, country + group by session_id, visit_id, country ) as t; `; } diff --git a/src/queries/analytics/sessions/getWebsiteSessions.ts b/src/queries/analytics/sessions/getWebsiteSessions.ts index b7c4f928..2b22a0c8 100644 --- a/src/queries/analytics/sessions/getWebsiteSessions.ts +++ b/src/queries/analytics/sessions/getWebsiteSessions.ts @@ -44,11 +44,11 @@ async function clickhouseQuery(websiteId: string, filters: QueryFilters, pagePar country, subdivision1, city, - min(created_at) as firstAt, - max(created_at) as lastAt, + min(min_time) as firstAt, + max(max_time) as lastAt, uniq(visit_id) as visits, - sumIf(1, event_type = 1) as views - from website_event + sumIf(views, event_type = 1) as views + from website_event_stats_hourly where website_id = {websiteId:UUID} ${dateQuery} ${filterQuery}