From d471d052178a6ca81403b8c26c73eda083939416 Mon Sep 17 00:00:00 2001
From: Mike Cao <mike@mikecao.com>
Date: Wed, 29 Mar 2023 23:08:26 -0700
Subject: [PATCH 1/4] Added config to open websites in new tab.

---
 .../pages/settings/websites/WebsiteSettings.js   | 16 +++++++---------
 .../pages/settings/websites/WebsitesTable.js     |  4 +++-
 2 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/components/pages/settings/websites/WebsiteSettings.js b/components/pages/settings/websites/WebsiteSettings.js
index dd8b1abe..5a55a36d 100644
--- a/components/pages/settings/websites/WebsiteSettings.js
+++ b/components/pages/settings/websites/WebsiteSettings.js
@@ -10,23 +10,21 @@ import TrackingCode from 'components/pages/settings/websites/TrackingCode';
 import ShareUrl from 'components/pages/settings/websites/ShareUrl';
 import useApi from 'hooks/useApi';
 import useMessages from 'hooks/useMessages';
+import useConfig from 'hooks/useConfig';
 
 export default function WebsiteSettings({ websiteId }) {
   const router = useRouter();
   const { formatMessage, labels, messages } = useMessages();
-  const [values, setValues] = useState(null);
-  const [tab, setTab] = useState('details');
+  const { openExternal } = useConfig();
   const { get, useQuery } = useApi();
   const { toast, showToast } = useToast();
   const { data, isLoading } = useQuery(
     ['website', websiteId],
-    () => {
-      if (websiteId) {
-        return get(`/websites/${websiteId}`);
-      }
-    },
-    { cacheTime: 0 },
+    () => get(`/websites/${websiteId}`),
+    { enabled: !!websiteId, cacheTime: 0 },
   );
+  const [values, setValues] = useState(null);
+  const [tab, setTab] = useState('details');
 
   const handleSave = data => {
     showToast({ message: formatMessage(messages.saved), variant: 'success' });
@@ -58,7 +56,7 @@ export default function WebsiteSettings({ websiteId }) {
           </Breadcrumbs>
         }
       >
-        <Link href={`/analytics/websites/${websiteId}`} target="_blank">
+        <Link href={`/websites/${websiteId}`} target={openExternal ? '_blank' : null}>
           <Button variant="primary">
             <Icon>
               <Icons.External />
diff --git a/components/pages/settings/websites/WebsitesTable.js b/components/pages/settings/websites/WebsitesTable.js
index 6f7290a1..27cc239e 100644
--- a/components/pages/settings/websites/WebsitesTable.js
+++ b/components/pages/settings/websites/WebsitesTable.js
@@ -13,9 +13,11 @@ import {
   Flexbox,
 } from 'react-basics';
 import useMessages from 'hooks/useMessages';
+import useConfig from 'hooks/useConfig';
 
 export default function WebsitesTable({ data = [] }) {
   const { formatMessage, labels } = useMessages();
+  const { openExternal } = useConfig();
 
   const columns = [
     { name: 'name', label: formatMessage(labels.name), style: { flex: 2 } },
@@ -48,7 +50,7 @@ export default function WebsitesTable({ data = [] }) {
                   <Text>{formatMessage(labels.edit)}</Text>
                 </Button>
               </Link>
-              <Link href={`/websites/${id}`}>
+              <Link href={`/websites/${id}`} target={openExternal ? '_blank' : null}>
                 <Button>
                   <Icon>
                     <Icons.External />

From a026fc97fcd2e74c74e34d329f05bf2b0506e99b Mon Sep 17 00:00:00 2001
From: Mike Cao <mike@mikecao.com>
Date: Wed, 29 Mar 2023 23:14:09 -0700
Subject: [PATCH 2/4] Prevent blank urls.

---
 pages/api/send.ts | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/pages/api/send.ts b/pages/api/send.ts
index f6c8a321..11e8f4c1 100644
--- a/pages/api/send.ts
+++ b/pages/api/send.ts
@@ -96,6 +96,10 @@ export default async (req: NextApiRequestCollect, res: NextApiResponse) => {
   let [referrerPath, referrerQuery] = referrer?.split('?') || [];
   let referrerDomain;
 
+  if (!urlPath) {
+    urlPath = '/';
+  }
+
   if (referrerPath.startsWith('http')) {
     const refUrl = new URL(referrer);
     referrerPath = refUrl.pathname;
@@ -104,7 +108,7 @@ export default async (req: NextApiRequestCollect, res: NextApiResponse) => {
   }
 
   if (process.env.REMOVE_TRAILING_SLASH) {
-    urlPath = urlPath.replace(/\/$/, '');
+    urlPath = urlPath.replace(/.+\/$/, '');
   }
 
   await saveEvent({

From 358519dddd05d374cc57b14678c7984a1d334295 Mon Sep 17 00:00:00 2001
From: Mike Cao <mike@mikecao.com>
Date: Wed, 29 Mar 2023 23:29:37 -0700
Subject: [PATCH 3/4] Removed filters from pages.

---
 components/metrics/PagesTable.js              | 24 +------------------
 components/metrics/WebsiteChart.module.css    |  2 +-
 .../analytics/pageview/getPageviewMetrics.ts  |  4 ++--
 .../analytics/session/getSessionMetrics.ts    |  4 ++--
 4 files changed, 6 insertions(+), 28 deletions(-)

diff --git a/components/metrics/PagesTable.js b/components/metrics/PagesTable.js
index ba9a3153..e7b7e77a 100644
--- a/components/metrics/PagesTable.js
+++ b/components/metrics/PagesTable.js
@@ -1,38 +1,16 @@
-import { useState } from 'react';
 import FilterLink from 'components/common/FilterLink';
-import FilterButtons from 'components/common/FilterButtons';
-import { urlFilter } from 'lib/filters';
 import MetricsTable from './MetricsTable';
-import { FILTER_COMBINED, FILTER_RAW } from 'lib/constants';
 import useMessages from 'hooks/useMessages';
 
-const filters = {
-  [FILTER_RAW]: null,
-  [FILTER_COMBINED]: urlFilter,
-};
-
-export default function PagesTable({ websiteId, showFilters, ...props }) {
-  const [filter, setFilter] = useState(FILTER_COMBINED);
+export default function PagesTable({ websiteId, ...props }) {
   const { formatMessage, labels } = useMessages();
 
-  const buttons = [
-    {
-      label: formatMessage(labels.filterCombined),
-      key: FILTER_COMBINED,
-    },
-    {
-      label: formatMessage(labels.filterRaw),
-      key: FILTER_RAW,
-    },
-  ];
-
   const renderLink = ({ x: url }) => {
     return <FilterLink id="url" value={url} />;
   };
 
   return (
     <>
-      {showFilters && <FilterButtons items={buttons} selectedKey={filter} onSelect={setFilter} />}
       <MetricsTable
         title={formatMessage(labels.pages)}
         type="url"
diff --git a/components/metrics/WebsiteChart.module.css b/components/metrics/WebsiteChart.module.css
index 3db8bc63..8d5acb06 100644
--- a/components/metrics/WebsiteChart.module.css
+++ b/components/metrics/WebsiteChart.module.css
@@ -24,12 +24,12 @@
   min-height: 90px;
   margin-bottom: 20px;
   background: var(--base50);
+  z-index: var(--z-index300);
 }
 
 .sticky {
   position: sticky;
   top: -1px;
-  z-index: 2;
 }
 
 .isSticky {
diff --git a/queries/analytics/pageview/getPageviewMetrics.ts b/queries/analytics/pageview/getPageviewMetrics.ts
index 214dbaab..16cad666 100644
--- a/queries/analytics/pageview/getPageviewMetrics.ts
+++ b/queries/analytics/pageview/getPageviewMetrics.ts
@@ -58,7 +58,7 @@ async function relationalQuery(
       ${filterQuery}
     group by 1
     order by 2 desc
-    limit 200`,
+    limit 100`,
     params,
   );
 }
@@ -93,7 +93,7 @@ async function clickhouseQuery(
       ${filterQuery}
     group by x
     order by y desc
-    limit 200`,
+    limit 100`,
     params,
   );
 }
diff --git a/queries/analytics/session/getSessionMetrics.ts b/queries/analytics/session/getSessionMetrics.ts
index 703ca871..011c099b 100644
--- a/queries/analytics/session/getSessionMetrics.ts
+++ b/queries/analytics/session/getSessionMetrics.ts
@@ -44,7 +44,7 @@ async function relationalQuery(
     )
     group by 1
     order by 2 desc
-    limit 200`,
+    limit 100`,
     params,
   );
 }
@@ -70,7 +70,7 @@ async function clickhouseQuery(
       ${filterQuery}
     group by x
     order by y desc
-    limit 200`,
+    limit 100`,
     params,
   );
 }

From 64b040c8653802e789e2adf1b7206c62c40e916f Mon Sep 17 00:00:00 2001
From: Mike Cao <mike@mikecao.com>
Date: Thu, 30 Mar 2023 09:44:04 -0700
Subject: [PATCH 4/4] Fixed query parameter filter.

---
 lib/clickhouse.ts                  | 4 ++--
 lib/filters.js                     | 2 +-
 pages/api/websites/[id]/metrics.ts | 3 ++-
 3 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/lib/clickhouse.ts b/lib/clickhouse.ts
index d9e6a02b..031f9319 100644
--- a/lib/clickhouse.ts
+++ b/lib/clickhouse.ts
@@ -190,8 +190,8 @@ function parseFilters(filters: any = {}, params: any = {}) {
 
 async function rawQuery(query, params = {}) {
   if (process.env.LOG_QUERY) {
-    log(query);
-    log(params);
+    log('QUERY:\n', query);
+    log('PARAMETERS:\n', params);
   }
 
   await connect();
diff --git a/lib/filters.js b/lib/filters.js
index 9681a802..8bc6229d 100644
--- a/lib/filters.js
+++ b/lib/filters.js
@@ -50,7 +50,7 @@ export const percentFilter = data => {
 export const paramFilter = data => {
   const map = data.reduce((obj, { x, y }) => {
     try {
-      const searchParams = new URLSearchParams(x.split('?')[1]);
+      const searchParams = new URLSearchParams(x);
 
       for (const [key, value] of searchParams) {
         if (!obj[key]) {
diff --git a/pages/api/websites/[id]/metrics.ts b/pages/api/websites/[id]/metrics.ts
index ad57790e..e6bae8f2 100644
--- a/pages/api/websites/[id]/metrics.ts
+++ b/pages/api/websites/[id]/metrics.ts
@@ -89,6 +89,7 @@ export default async (
     subdivision1,
     subdivision2,
     city,
+    query,
   } = req.query;
 
   if (req.method === 'GET') {
@@ -162,7 +163,7 @@ export default async (
         subdivision2: type !== 'subdivision2' ? subdivision2 : undefined,
         city: type !== 'city' ? city : undefined,
         eventUrl: type !== 'url' && table === 'event' ? url : undefined,
-        query: type === 'query' && table !== 'event' ? true : undefined,
+        query: type !== 'query' && table !== 'event' ? query : undefined,
       };
 
       const data = await getPageviewMetrics(websiteId, {