bugfix: only log search url on search route (#33)

* feat: udpate search route string

Signed-off-by: Patrick Erichsen <patrick.a.erichsen@gmail.com>

* fix: only log search url

Signed-off-by: Patrick Erichsen <patrick.a.erichsen@gmail.com>

src/content/App.tsx

@@ -1,9 +1,9 @@
 import { useState } from 'react';
 import { useUrlChangeChrome } from './hooks/useUrlChangeChrome';
 import { OffiePortals } from './components/OffiePortals';
-import { isOffiePage } from './utils';
+import { isOffiePage, isSearchUrl } from './utils';
 import './App.css';
-import { logUrlChange } from '../utils';
+import { logSearchUrl } from '../utils';
 
 export const App = (): JSX.Element | null => {
     const [isVisible, setIsVisible] = useState<boolean>(false);
@@ -19,8 +19,8 @@ export const App = (): JSX.Element | null => {
         /**
          * Only log URL changes if we are on the homes search page
          */
-        if (newIsVisible) {
-            logUrlChange(newUrl);
+        if (newIsVisible && isSearchUrl(newUrl)) {
+            logSearchUrl(newUrl);
         }
     });
 

src/content/utils/airbnb.ts

@@ -274,6 +274,12 @@ export const getMappedSearchFilters = (
     }, {});
 };
 
+export const isSearchUrl = (url: string): boolean => {
+    const { pathname } = new URL(url);
+
+    return pathname.includes(SEARCH_ROUTE);
+};
+
 export const isOffiePage = (url: string): boolean => {
     const { pathname } = new URL(url);
 

src/utils/analytics.ts

@@ -47,7 +47,7 @@ export const tryIdentifyUserFromUrlParam = (url: string): void => {
     }
 };
 
-export const logUrlChange = (url: string): void => {
+export const logSearchUrl = (url: string): void => {
     const { search, pathname } = new URL(url);
 
     const decodedPathname = decodeURIComponent(pathname);