@playpilot/tpi 8.40.0 → 8.40.1

package/src/lib/types/config.d.ts

@@ -139,5 +139,4 @@ export type ConfigResponse = {
   explore_navigation_insert_position?: InsertPosition
   explore_insert_cta_in_tpi?: boolean
   explore_custom_style?: string
-  explore_use_router?: boolean
 }

package/src/routes/components/Debugger.svelte

@@ -3,7 +3,7 @@
   import { fetchLinkInjections } from '$lib/api/externalPages'
   import { Language } from '$lib/enums/Language'
   import { SplitTest } from '$lib/enums/SplitTest'
-  import { destroyExplore, insertExplore, useExploreRouter } from '$lib/explore'
+  import { destroyExplore, insertExplore } from '$lib/explore'
   import { getPageTextAndElements } from '$lib/injectionElements'
   import { getLanguage } from '$lib/language'
   import { getFullUrlPath } from '$lib/url'
@@ -217,14 +217,6 @@
       <button onclick={replaceWithBetaScript}>Use beta script</button>
     {/if}
 
-    {#if !useExploreRouter()}
-      <button onclick={(() => {
-        destroyExplore()
-        window.PlayPilotLinkInjections.config!.explore_use_router = !useExploreRouter()
-        insertExplore()
-      })}>Toggle new explore</button>
-    {/if}
-
     <hr />
 
     <div>

package/src/routes/components/Explore/ExploreLayout.svelte

@@ -1,7 +1,7 @@
 <script lang="ts">
   import type { ExploreFilter } from '$lib/types/filter'
   import { heading } from '$lib/actions/heading'
-  import { exploreParentSelector, useExploreRouter } from '$lib/explore'
+  import { exploreParentSelector } from '$lib/explore'
   import { t } from '$lib/localization'
   import { track } from '$lib/tracking'
   import { TrackingEvent } from '$lib/enums/TrackingEvent'
@@ -57,28 +57,20 @@
         <!-- eslint-disable-next-line svelte/no-at-html-tags -->
         {@html t('Streaming Guide Disclaimer')}
       </p>
-
-      {#if !useExploreRouter()}
-        <p class="description">
-          {t('Streaming Guide Description')}
-        </p>
-      {/if}
     </div>
   </div>
 
   <Search oninput={(query) => searchQuery = query} />
 
-  {#if useExploreRouter()}
-    <div class="filter">
-      <Filter
-        {filter}
-        {searchQuery}
-        limit={clientWidth < 500}
-        showSorting={!searchQuery}
-        onchange={() => navigate('results')}
-        onempty={() => navigate('home')} />
-    </div>
-  {/if}
+  <div class="filter">
+    <Filter
+      {filter}
+      {searchQuery}
+      limit={clientWidth < 500}
+      showSorting={!searchQuery}
+      onchange={() => navigate('results')}
+      onempty={() => navigate('home')} />
+  </div>
 
   {#await isUserRegionSupported() then isSupported}
     {#if !isSupported}

package/src/routes/components/Explore/ExploreRouter.svelte

@@ -1,7 +1,6 @@
 <script lang="ts">
   import type { ExploreRoute } from '$lib/types/explore'
   import type { ExploreFilter } from '$lib/types/filter'
-  import { useExploreRouter } from '$lib/explore'
   import { track } from '$lib/tracking'
   import { TrackingEvent } from '$lib/enums/TrackingEvent'
   import ExploreHome from './Routes/ExploreHome.svelte'
@@ -11,29 +10,24 @@
   import ExploreModal from './Routes/ExploreModal.svelte'
 
   const routes: ExploreRoute[] = [
+    {
+      key: 'home',
+      component: ExploreHome,
+    },
     {
       key: 'results',
       component: ExploreResults,
     },
-  ]
-
-  if (useExploreRouter()) {
-    routes.unshift({
-      key: 'home',
-      component: ExploreHome,
-    })
-
-    routes.push({
+    {
       key: 'title',
       component: ExploreTitle,
-    })
-
-    routes.push({
+    },
+    {
       key: 'modal',
       component: ExploreHome,
       appendComponent: ExploreModal,
-    })
-  }
+    }
+  ]
 
   const initialRouteKey = getCurrentRouteParam() || routes[0].key
 

package/src/routes/components/Explore/Routes/ExploreHome.svelte

@@ -32,7 +32,7 @@
   }
 </script>
 
-<svelte:window {onscroll} bind:innerWidth={windowWidth} />
+<svelte:window bind:innerWidth={windowWidth} />
 
 <div data-testid="explore-home"></div>
 

package/src/routes/components/Rails/TitlesRail.svelte

@@ -8,7 +8,6 @@
   import { exploreModalUrl } from '$lib/routes'
   import { generateRandomHash } from '$lib/hash'
   import { getFirstTitleWithAvailableTrailer } from '$lib/trailer'
-  import { useExploreRouter } from '$lib/explore'
   import { getContext, onMount } from 'svelte'
   import type TinySlider from 'svelte-tiny-slider'
 
@@ -92,11 +91,7 @@
       return
     }
 
-    if (useExploreRouter()) {
-      openModal({ event, type: 'titles-rail', data: titles as TitleData[], returnToTitle, props: { initialIndex: index } })
-    } else {
-      openModal({ event, data: title, returnToTitle })
-    }
+    openModal({ event, type: 'titles-rail', data: titles as TitleData[], returnToTitle, props: { initialIndex: index } })
   }
 
   async function expandFirstAvailableTrailer(): Promise<void> {

package/src/routes/explore/+page.svelte

@@ -6,9 +6,8 @@
    */
 
   import { browser } from '$app/environment'
-  import { destroyExplore, insertExplore, insertExploreIntoNavigation, useExploreRouter } from '$lib/explore'
+  import { insertExplore, insertExploreIntoNavigation } from '$lib/explore'
   import Debugger from '../components/Debugger.svelte'
-
   if (browser) {
     // @ts-ignore
     window.PlayPilotLinkInjections = {
@@ -18,7 +17,6 @@
         explore_navigation_label: 'Streaming guide (inserted)',
         explore_navigation_path: '/explore',
         explore_custom_style: 'main { border: 2px solid white }',
-        explore_use_router: true,
         open_tpi_links_in_explore: true,
       },
       require_consent: false,
@@ -34,11 +32,7 @@
   <title>PlayPilot Link Injections - Explore</title>
 </svelte:head>
 
-<button onclick={(() => {
-  destroyExplore()
-  window.PlayPilotLinkInjections.config!.explore_use_router = !useExploreRouter()
-  insertExplore()
-})}>Toggle new explore</button>
+
 
 <main>
   <!--

package/src/tests/lib/api/ads.test.js

@@ -72,6 +72,14 @@ describe('$lib/api/ads', () => {
       await expect(async () => await fetchAds()).rejects.toThrowError()
       expect(track).toHaveBeenCalledWith(TrackingEvent.AdsFetchFailed, null, { status: 505 })
     })
+
+    it('Should include set region', async () => {
+      window.PlayPilotLinkInjections.region = 'uk'
+
+      await fetchAds()
+
+      expect(api).toHaveBeenCalledWith(expect.stringContaining('&region=uk'))
+    })
   })
 
   describe('getFirstAdOfType', () => {

package/svelte.config.js

@@ -1,23 +1,32 @@
-import adapter from '@sveltejs/adapter-auto'
-import { sveltePreprocess  } from 'svelte-preprocess'
-
-/** @type {import('@sveltejs/kit').Config} */
-const config = {
-  preprocess: sveltePreprocess({
-    scss: {
-      includePaths: ['src'],
-      prependData: `
-        @use "src/lib/scss/_functions.scss" as *;
-        @use "src/lib/scss/_mixins.scss" as *;
-      `,
-    },
-  }),
-  kit: {
-    // adapter-auto only supports some environments, see https://svelte.dev/docs/kit/adapter-auto for a list.
-    // If your environment is not supported, or you settled on a specific environment, switch out the adapter.
-    // See https://svelte.dev/docs/kit/adapters for more information about adapters.
-    adapter: adapter(),
-  },
-}
-
-export default config
+import adapter from '@sveltejs/adapter-auto'
+import { sveltePreprocess } from 'svelte-preprocess'
+
+/** @type {import('@sveltejs/kit').Config} */
+const config = {
+  preprocess: sveltePreprocess({
+    scss: {
+      includePaths: ['src'],
+      prependData: `
+        @use "src/lib/scss/_functions.scss" as *;
+        @use "src/lib/scss/_mixins.scss" as *;
+      `,
+    },
+  }),
+  kit: {
+    // adapter-auto only supports some environments, see https://svelte.dev/docs/kit/adapter-auto for a list.
+    // If your environment is not supported, or you settled on a specific environment, switch out the adapter.
+    // See https://svelte.dev/docs/kit/adapters for more information about adapters.
+    adapter: adapter(),
+  },
+  compilerOptions: {
+    warningFilter: (warning) => {
+      const ignore = [
+        'state_referenced_locally',
+      ]
+
+      return !ignore.includes(warning.code)
+    },
+  },
+}
+
+export default config

package/vite.config.js

@@ -1,43 +1,43 @@
-import path from 'path'
-import * as dotenv from 'dotenv'
-import { defineConfig } from 'vitest/config'
-import { sveltekit } from '@sveltejs/kit/vite'
-import packageJson from './package.json'
-
-dotenv.config({ path: '.env' })
-
-export default defineConfig(() => ({
-  plugins: [
-    sveltekit(),
-  ],
-
-  server: {
-    port: 3000,
-    allowedHosts: ['engage-moderate-happy-davis.trycloudflare.com'],
-  },
-
-  test: {
-    environment: 'happy-dom',
-    environmentOptions: {
-      happyDOM: {
-        settings: {
-          disableJavaScriptFileLoading: true,
-        },
-      },
-    },
-    include: ['src/**/*.{test,spec}.{js,ts}'],
-    setupFiles: ['src/tests/setup.js'],
-  },
-
-  resolve: {
-    conditions: process.env.VITEST ? ['browser'] : [],
-    alias: {
-      '$lib': path.resolve(__dirname, './src/lib'),
-    },
-  },
-
-  define: {
-    __SCRIPT_VERSION__: JSON.stringify(packageJson.version),
-    __IS_EDITORIAL_SCRIPT__: true,
-  },
-}))
+import path from 'path'
+import * as dotenv from 'dotenv'
+import { defineConfig } from 'vitest/config'
+import { sveltekit } from '@sveltejs/kit/vite'
+import packageJson from './package.json'
+
+dotenv.config({ path: '.env' })
+
+export default defineConfig(() => ({
+  plugins: [
+    sveltekit(),
+  ],
+
+  server: {
+    port: 3000,
+    allowedHosts: ['engage-moderate-happy-davis.trycloudflare.com'],
+  },
+
+  test: {
+    environment: 'happy-dom',
+    environmentOptions: {
+      happyDOM: {
+        settings: {
+          disableJavaScriptFileLoading: true,
+        },
+      },
+    },
+    include: ['src/**/*.{test,spec}.{js,ts}'],
+    setupFiles: ['src/tests/setup.js'],
+  },
+
+  resolve: {
+    conditions: process.env.VITEST ? ['browser'] : [],
+    alias: {
+      '$lib': path.resolve(__dirname, './src/lib'),
+    },
+  },
+
+  define: {
+    __SCRIPT_VERSION__: JSON.stringify(packageJson.version),
+    __IS_EDITORIAL_SCRIPT__: true,
+  },
+}))