@playpilot/tpi 8.27.1 → 8.28.0

package/src/routes/components/Participant.svelte

@@ -1,160 +1,160 @@
-<script lang="ts">
-  import { onMount } from 'svelte'
-  import { heading } from '$lib/actions/heading'
-  import { fetchTitlesForParticipant } from '$lib/api/participants'
-  import { openModal } from '$lib/modal'
-  import type { ParticipantData } from '$lib/types/participant'
-  import type { TitleData } from '$lib/types/title'
-  import ListTitle from './ListTitle.svelte'
-  import { t } from '$lib/localization'
-  import { isRetargetingAllowed } from '$lib/retargeting'
-  import { trackViaPixel } from '@playpilot/retargeting-tracking'
-  import { MetaEvent } from '$lib/enums/TrackingEvent'
-
-  interface Props {
-    participant: ParticipantData
-    small?: boolean
-  }
-
-  const { participant, small = false }: Props = $props()
-
-  const { name, birth_date, death_date } = $derived(participant)
-
-  const pageSize = 30
-
-  let titles: TitleData[] = $state([])
-  let page = $state(1)
-  let hasMorePages = $state(true)
-  let loading = $state(true)
-
-  onMount(loadMore)
-
-  if (isRetargetingAllowed()) trackViaPixel(MetaEvent.ParticipantView, { participant: participant.name })
-
-  function formatDate(dateString: string): string {
-    const date = new Date(dateString)
-    return date.toLocaleDateString(undefined, { year: 'numeric', month: 'long', day: 'numeric' })
-  }
-
-  async function loadMore(): Promise<void> {
-    loading = true
-
-    try {
-      const results = await fetchTitlesForParticipant(participant, { page })
-
-      titles = [...titles, ...results]
-      hasMorePages = results?.length >= pageSize
-    } catch {
-      hasMorePages = false
-    } finally {
-      loading = false
-      page += 1
-    }
-  }
-</script>
-
-<div class="header" class:small>
-  <div class="heading" use:heading={2} id="heading">{name}</div>
-
-  {#if birth_date}
-    <p class="dates">
-      {t('Born On')} <strong>{formatDate(birth_date)}</strong>{#if death_date}, {t('Died On')} <strong>{formatDate(death_date)}</strong>{/if}
-    </p>
-  {/if}
-</div>
-
-<div class="content">
-  {#if !small}
-    <div class="heading subheading" use:heading={3} id="credits">{t('Credits')}</div>
-  {/if}
-
-  <div class="list">
-    {#each titles as title}
-      <ListTitle {title} compact={small} onclick={(event) => openModal({ event, data: title })} />
-    {/each}
-
-    {#if loading}
-      {#each { length: 6 }}
-        <div class="skeleton" data-testid="skeleton"></div>
-      {/each}
-    {:else if hasMorePages}
-      <button class="more" onclick={loadMore}>{t('Show More')}</button>
-    {/if}
-  </div>
-</div>
-
-<style lang="scss">
-  .header {
-    padding: margin(4) margin(1) margin(2);
-    font-family: theme(detail-font-family, font-family);
-    font-weight: theme(detail-font-weight, normal);
-    font-size: theme(detail-font-size, font-size-base);
-    line-height: theme(participant-description-line-height, normal);
-    color: theme(detail-text-color, text-color);
-
-    &.small {
-      padding: margin(1);
-    }
-  }
-
-  .heading {
-    margin: 0;
-    font-family: theme(detail-title-font-family, font-family);
-    font-weight: theme(detail-title-font-weight, lighter);
-    font-size: theme(detail-title-font-size, margin(1.5));
-    line-height: normal;
-    font-style: theme(detail-title-font-style, normal);
-
-    &.subheading {
-      margin: 0 0 margin(0.5);
-      font-size: theme(detail-title-small-font-size, margin(1.25));
-    }
-  }
-
-  .dates {
-    margin: margin(0.5) 0 0;
-  }
-
-  .content {
-    padding: 0 margin(1) margin(1);
-    color: theme(detail-text-color, text-color);
-    font-family: theme(detail-font-family, font-family);
-    font-weight: theme(detail-font-weight, normal);
-    font-size: theme(detail-font-size, font-size-base);
-    line-height: normal;
-    font-style: normal;
-
-    ::view-transition-old(content),
-    ::view-transition-new(content) {
-      height: 100%;
-    }
-  }
-
-  .list {
-    display: flex;
-    flex-direction: column;
-    gap: margin(0.5);
-  }
-
-  .skeleton {
-    min-height: margin(7.25);
-    border-radius: theme(playlink-border-radius, border-radius);
-    background: theme(detail-background-light, lighter);
-  }
-
-  .more {
-    cursor: pointer;
-    appearance: none;
-    padding: margin(0.5) margin(1);
-    border: 0;
-    border-radius: theme(participants-load-more-border-radius, border-radius);
-    background: theme(participants-load-more-background, content);
-    color: inherit;
-    font-size: inherit;
-    font-family: inherit;
-
-    &:hover {
-      filter: brightness(1.2);
-    }
-  }
-</style>
+<script lang="ts">
+  import { onMount } from 'svelte'
+  import { heading } from '$lib/actions/heading'
+  import { fetchTitlesForParticipant } from '$lib/api/participants'
+  import { openModal } from '$lib/modal'
+  import type { ParticipantData } from '$lib/types/participant'
+  import type { TitleData } from '$lib/types/title'
+  import ListTitle from './ListTitle.svelte'
+  import { t } from '$lib/localization'
+  import { isRetargetingAllowed } from '$lib/retargeting'
+  import { trackViaPixel } from '@playpilot/retargeting-tracking'
+  import { MetaEvent } from '$lib/enums/TrackingEvent'
+
+  interface Props {
+    participant: ParticipantData
+    small?: boolean
+  }
+
+  const { participant, small = false }: Props = $props()
+
+  const { name, birth_date, death_date } = $derived(participant)
+
+  const pageSize = 30
+
+  let titles: TitleData[] = $state([])
+  let page = $state(1)
+  let hasMorePages = $state(true)
+  let loading = $state(true)
+
+  onMount(loadMore)
+
+  if (isRetargetingAllowed()) trackViaPixel(MetaEvent.ParticipantView, { participant: participant.name })
+
+  function formatDate(dateString: string): string {
+    const date = new Date(dateString)
+    return date.toLocaleDateString(undefined, { year: 'numeric', month: 'long', day: 'numeric' })
+  }
+
+  async function loadMore(): Promise<void> {
+    loading = true
+
+    try {
+      const results = await fetchTitlesForParticipant(participant, { page })
+
+      titles = [...titles, ...results]
+      hasMorePages = results?.length >= pageSize
+    } catch {
+      hasMorePages = false
+    } finally {
+      loading = false
+      page += 1
+    }
+  }
+</script>
+
+<div class="header" class:small>
+  <div class="heading" use:heading={2} id="heading">{name}</div>
+
+  {#if birth_date}
+    <p class="dates">
+      {t('Born On')} <strong>{formatDate(birth_date)}</strong>{#if death_date}, {t('Died On')} <strong>{formatDate(death_date)}</strong>{/if}
+    </p>
+  {/if}
+</div>
+
+<div class="content">
+  {#if !small}
+    <div class="heading subheading" use:heading={3} id="credits">{t('Credits')}</div>
+  {/if}
+
+  <div class="list">
+    {#each titles as title}
+      <ListTitle {title} compact={small} onclick={(event) => openModal({ event, data: title })} />
+    {/each}
+
+    {#if loading}
+      {#each { length: 6 }}
+        <div class="skeleton" data-testid="skeleton"></div>
+      {/each}
+    {:else if hasMorePages}
+      <button class="more" onclick={loadMore}>{t('Show More')}</button>
+    {/if}
+  </div>
+</div>
+
+<style lang="scss">
+  .header {
+    padding: margin(4) margin(1) margin(2);
+    font-family: theme(detail-font-family, font-family);
+    font-weight: theme(detail-font-weight, normal);
+    font-size: theme(detail-font-size, font-size-base);
+    line-height: theme(participant-description-line-height, normal);
+    color: theme(detail-text-color, text-color);
+
+    &.small {
+      padding: margin(1);
+    }
+  }
+
+  .heading {
+    margin: 0;
+    font-family: theme(detail-title-font-family, font-family);
+    font-weight: theme(detail-title-font-weight, lighter);
+    font-size: theme(detail-title-font-size, margin(1.5));
+    line-height: normal;
+    font-style: theme(detail-title-font-style, normal);
+
+    &.subheading {
+      margin: 0 0 margin(0.5);
+      font-size: theme(detail-title-small-font-size, margin(1.25));
+    }
+  }
+
+  .dates {
+    margin: margin(0.5) 0 0;
+  }
+
+  .content {
+    padding: 0 margin(1) margin(1);
+    color: theme(detail-text-color, text-color);
+    font-family: theme(detail-font-family, font-family);
+    font-weight: theme(detail-font-weight, normal);
+    font-size: theme(detail-font-size, font-size-base);
+    line-height: normal;
+    font-style: normal;
+
+    ::view-transition-old(content),
+    ::view-transition-new(content) {
+      height: 100%;
+    }
+  }
+
+  .list {
+    display: flex;
+    flex-direction: column;
+    gap: margin(0.5);
+  }
+
+  .skeleton {
+    min-height: margin(7.25);
+    border-radius: theme(playlink-border-radius, border-radius);
+    background: theme(detail-background-light, lighter);
+  }
+
+  .more {
+    cursor: pointer;
+    appearance: none;
+    padding: margin(0.5) margin(1);
+    border: 0;
+    border-radius: theme(participants-load-more-border-radius, border-radius);
+    background: theme(participants-load-more-background, content);
+    color: inherit;
+    font-size: inherit;
+    font-family: inherit;
+
+    &:hover {
+      filter: brightness(1.2);
+    }
+  }
+</style>

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

@@ -35,7 +35,7 @@
 
 <button onclick={(() => {
   destroyExplore()
-  window.PlayPilotLinkInjections.config.explore_use_router = !useExploreRouter()
+  window.PlayPilotLinkInjections.config!.explore_use_router = !useExploreRouter()
   insertExplore()
 })}>Toggle new explore</button>
 
@@ -73,4 +73,3 @@
     background: theme(light);
   }
 </style>
-

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

@@ -40,7 +40,7 @@ describe('$lib/api/externalPages', () => {
 
       expect(response).toBe('Some response')
       expect(api).toHaveBeenCalledWith(
-        '/external-pages/?api-token=a&include_title_details=true&language=en-US&url=https://some-url',
+        '/external-pages/?api-token=a&include_title_details=true&include_participant_details=true&language=en-US&url=https://some-url',
         expect.objectContaining({
           method: 'GET',
         }),

package/src/tests/lib/explore.test.js

@@ -120,7 +120,7 @@ describe('explore.js', () => {
 
     it('Should not insert if config object is present but explore_navigation_selector is not given', () => {
       window.PlayPilotLinkInjections.config = {
-        explore_navigation_selector: null,
+        explore_navigation_selector: '',
       }
 
       expect(insertExploreIntoNavigation()).toBe(false)
@@ -136,11 +136,12 @@ describe('explore.js', () => {
       expect(insertExploreIntoNavigation()).toBe(false)
     })
 
-    it('Should insert if explore_navigation_selector is given and selector is valid', () => {
+    it('Should insert if explore_navigation_selector is given and selector and path are valid', () => {
       document.body.innerHTML = '<nav><a></a></nav>'
 
       window.PlayPilotLinkInjections.config = {
         explore_navigation_selector: 'a',
+        explore_navigation_path: '/explore',
       }
 
       expect(insertExploreIntoNavigation()).toBe(true)
@@ -165,6 +166,7 @@ describe('explore.js', () => {
       window.PlayPilotLinkInjections.config = {
         explore_navigation_selector: 'a',
         explore_navigation_label: 'Some label',
+        explore_navigation_path: '/explore',
       }
 
       insertExploreIntoNavigation()

package/src/tests/lib/injection.test.js

@@ -984,7 +984,7 @@ describe('injection.ts', () => {
         injectLinksInDocument(elements, { aiInjections: [injection], manualInjections: [] })
 
         const link = /** @type {HTMLAnchorElement} */ (document.querySelector('a'))
-        expect(link.href).toBe(window.PlayPilotLinkInjections.config.explore_navigation_path + `?route=modal&sid=${injection.title_details?.sid}`)
+        expect(link.href).toBe(window.PlayPilotLinkInjections.config?.explore_navigation_path + `?route=modal&sid=${injection.title_details?.sid}`)
         expect(link.target).not.toBeTruthy()
       })
 

package/src/tests/lib/retargeting.test.js

@@ -26,6 +26,7 @@ describe('retargeting.ts', () => {
   describe('isPixelAllow', () => {
     it('Should return false if no config object is set', () => {
       vi.mocked(hasConsentedTo).mockImplementation(() => true)
+      // @ts-ignore
       window.PlayPilotLinkInjections.config = null
 
       expect(isRetargetingAllowed()).toBe(false)
@@ -33,6 +34,7 @@ describe('retargeting.ts', () => {
 
     it('Should return false if config object is without allow_retargeting_pixels', () => {
       vi.mocked(hasConsentedTo).mockImplementation(() => true)
+      // @ts-ignore
       window.PlayPilotLinkInjections.config = { something: true }
 
       expect(isRetargetingAllowed()).toBe(false)

package/src/tests/routes/components/Playlinks/Playlink.test.js

@@ -118,6 +118,7 @@ describe('Playlink.svelte', () => {
   })
 
   it('Should render as link when user has not consented to affiliate but require_affiliate_consent is false', () => {
+    // @ts-ignore
     window.PlayPilotLinkInjections.config.require_affiliate_consent = false
 
     vi.mocked(hasConsentedTo).mockImplementation(() => false)
@@ -129,6 +130,7 @@ describe('Playlink.svelte', () => {
   })
 
   it('Should render as link when user has not consented to affiliate and require_affiliate_consent is true', () => {
+    // @ts-ignore
     window.PlayPilotLinkInjections.config.require_affiliate_consent = true
 
     vi.mocked(hasConsentedTo).mockImplementation(() => false)

package/src/tests/setup.js

@@ -1,108 +1,108 @@
-import { cleanup } from '@testing-library/svelte'
-import { afterEach, beforeAll, beforeEach, vi } from 'vitest'
-import { fakeFetch } from './helpers'
-
-vi.mock('svelte/transition')
-
-// https://github.com/jsdom/jsdom/issues/3429#issuecomment-1936128876
-const mockAnimations = () => {
-  Element.prototype.animate ??= vi.fn().mockReturnValue({
-    onfinish: null,
-    finished: Promise.resolve(),
-    cancel: vi.fn(),
-    startTime: null,
-    currentTime: null,
-  })
-
-  Element.prototype.getAnimations ??= vi.fn().mockReturnValue([])
-}
-
-const mockLocalStorage = () => {
-  /** @type {Record<string, string>} */
-  let store = {}
-
-  // @ts-ignore
-  ;(window).localStorage = {
-    getItem: function (key) {
-      return store[key] || null
-    },
-    setItem: function (key, value) {
-      store[key] = value.toString()
-    },
-    removeItem: function (key) {
-      delete store[key]
-    },
-    clear: function () {
-      store = {}
-    },
-  }
-}
-
-const mockSessionStorage = () => {
-  /** @type {Record<string, string>} */
-  let store = {}
-
-  // @ts-ignore
-  ;(window).sessionStorage = {
-    getItem: function (key) {
-      return store[key] || null
-    },
-    setItem: function (key, value) {
-      store[key] = value.toString()
-    },
-    removeItem: function (key) {
-      delete store[key]
-    },
-    clear: function () {
-      store = {}
-    },
-  }
-}
-
-export const mockIntersectionObserver = () => {
-  // @ts-ignore
-  global.IntersectionObserver = vi.fn((callback) => ({
-    disconnect: vi.fn(),
-    observe: vi.fn(() => callback([{ isIntersecting: true }])),
-    unobserve: vi.fn(),
-  }))
-}
-
-beforeAll(() => {
-  fakeFetch()
-  mockLocalStorage()
-  mockSessionStorage()
-  mockIntersectionObserver()
-})
-
-beforeEach(() => {
-  // @ts-ignore
-  window.PlayPilotLinkInjections = { token: 'some-token', require_consent: false }
-
-  // Enable all consent
-  // @ts-ignore
-  window.__tcfapi = (command, _version, callback) => {
-    if (command !== 'addEventListener') return
-
-    setTimeout(() => callback({
-      purpose: { consents: { 1: true, 2: true, 7: true, 8: true, 9: true, 10: true } },
-      eventStatus: 'tcloaded',
-    }, true), 0)
-
-    return 1
-  }
-
-  // Reset cookies
-  document.cookie.split(';').forEach((cookie) => document.cookie = cookie.replace(/^ +/, '').replace(/=.*/, '=;'))
-
-  window.HTMLElement.prototype.scrollIntoView = vi.fn()
-
-  localStorage.clear()
-  sessionStorage.clear()
-
-  mockAnimations()
-})
-
-afterEach(() => {
-  cleanup()
-})
+import { cleanup } from '@testing-library/svelte'
+import { afterEach, beforeAll, beforeEach, vi } from 'vitest'
+import { fakeFetch } from './helpers'
+
+vi.mock('svelte/transition')
+
+// https://github.com/jsdom/jsdom/issues/3429#issuecomment-1936128876
+const mockAnimations = () => {
+  Element.prototype.animate ??= vi.fn().mockReturnValue({
+    onfinish: null,
+    finished: Promise.resolve(),
+    cancel: vi.fn(),
+    startTime: null,
+    currentTime: null,
+  })
+
+  Element.prototype.getAnimations ??= vi.fn().mockReturnValue([])
+}
+
+const mockLocalStorage = () => {
+  /** @type {Record<string, string>} */
+  let store = {}
+
+  // @ts-ignore
+  ;(window).localStorage = {
+    getItem: function (key) {
+      return store[key] || null
+    },
+    setItem: function (key, value) {
+      store[key] = value.toString()
+    },
+    removeItem: function (key) {
+      delete store[key]
+    },
+    clear: function () {
+      store = {}
+    },
+  }
+}
+
+const mockSessionStorage = () => {
+  /** @type {Record<string, string>} */
+  let store = {}
+
+  // @ts-ignore
+  ;(window).sessionStorage = {
+    getItem: function (key) {
+      return store[key] || null
+    },
+    setItem: function (key, value) {
+      store[key] = value.toString()
+    },
+    removeItem: function (key) {
+      delete store[key]
+    },
+    clear: function () {
+      store = {}
+    },
+  }
+}
+
+export const mockIntersectionObserver = () => {
+  // @ts-ignore
+  global.IntersectionObserver = vi.fn((callback) => ({
+    disconnect: vi.fn(),
+    observe: vi.fn(() => callback([{ isIntersecting: true }])),
+    unobserve: vi.fn(),
+  }))
+}
+
+beforeAll(() => {
+  fakeFetch()
+  mockLocalStorage()
+  mockSessionStorage()
+  mockIntersectionObserver()
+})
+
+beforeEach(() => {
+  // @ts-ignore
+  window.PlayPilotLinkInjections = { token: 'some-token', require_consent: false }
+
+  // Enable all consent
+  // @ts-ignore
+  window.__tcfapi = (command, _version, callback) => {
+    if (command !== 'addEventListener') return
+
+    setTimeout(() => callback({
+      purpose: { consents: { 1: true, 2: true, 7: true, 8: true, 9: true, 10: true } },
+      eventStatus: 'tcloaded',
+    }, true), 0)
+
+    return 1
+  }
+
+  // Reset cookies
+  document.cookie.split(';').forEach((cookie) => document.cookie = cookie.replace(/^ +/, '').replace(/=.*/, '=;'))
+
+  window.HTMLElement.prototype.scrollIntoView = vi.fn()
+
+  localStorage.clear()
+  sessionStorage.clear()
+
+  mockAnimations()
+})
+
+afterEach(() => {
+  cleanup()
+})