@playpilot/tpi 8.23.0 → 8.23.2

package/events.md

@@ -126,6 +126,7 @@ Event | Action | Info | Payload
 `venus_title_rail_modal_view` | _Fires any time a title is clicked in a rail, opening the rail modal_ | | `rail` (heading key of the rail)
 `venus_title_rail_expand_click` | _Fires any time a title is expanded in a rail directly via a click_ | Does not fire when a title opens automatically on load or navigate | `Title`, `rail` (heading key of the rail)
 `venus_title_rail_set_index` | _Fires when navigating in the titles rail modal, either via arrows, swipe, or clicking titles_ | | `Title`, `index` (index of the new position of the slider)
+`venus_title_reel_set_index` | _Fires when navigating in the titles reel modal, swipe, or clicking titles_ | | `Title`, `index` (index of the new position of the slider)
 `venus_navigate` | _Fires when navigating on the explore page_ | Does not fire on initial load | `route` (the key of the given route)
 `venus_any_click` | _Fires on any click anywhere_ | | `selector` (parent > child selector of the clicked element), `text` (direct text content of the clicked element, limited to 30 characters)
 `venus_scroll_distance` | _Fires on increments of scroll distance_ | 25%, 50%, 75%, 100% | `scroll_percentage` (percentage of the explore element the user has seen)

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@playpilot/tpi",
-	"version": "8.23.0",
+	"version": "8.23.2",
 	"type": "module",
 	"scripts": {
 		"dev": "vite dev",

package/src/lib/enums/TrackingEvent.ts

@@ -79,6 +79,7 @@ export const TrackingEvent = {
   ExploreTitleRailModalView: 'venus_title_rail_modal_view',
   ExploreTitleRailExpandClick: 'venus_title_rail_expand_click',
   ExploreTitleRailSetIndex: 'venus_title_rail_set_index',
+  ExploreTitleReelSetIndex: 'venus_title_reel_set_index',
   ExploreNavigate: 'venus_navigate',
   ExploreAnyClick: 'venus_any_click',
   ExploreScrollDistance: 'venus_scroll_distance',

package/src/lib/splitTest.ts

@@ -1,4 +1,3 @@
-import { hasConsentedTo } from './consent'
 import { TrackingEvent } from './enums/TrackingEvent'
 import { track } from './tracking'
 
@@ -15,8 +14,6 @@ export const sessionStoragePrefix = 'playpilot_split_test_'
  * The identifier is saved on the window object so that it is consistent across this session.
  */
 export function getSplitTestIdentifier({ key }: SplitTest): number {
-  if (!hasConsentedTo('tracking')) return 0
-
   const windowIdentifier = window.PlayPilotLinkInjections?.split_test_identifiers?.[key]
   if (windowIdentifier !== undefined) return windowIdentifier
 
@@ -25,7 +22,7 @@ export function getSplitTestIdentifier({ key }: SplitTest): number {
   let storedIdentifier = sessionStorage.getItem(sessionStoragePrefix + key)
 
   if (!storedIdentifier) {
-    const randomIdentifier = Math.random().toString()
+    const randomIdentifier = (Math.floor(Math.random() * 100) / 100).toString()
     sessionStorage.setItem(sessionStoragePrefix + key, randomIdentifier)
     storedIdentifier = randomIdentifier
   }
@@ -53,8 +50,6 @@ export function getSplitTestVariantName(test: SplitTest): string {
 }
 
 export function isInSplitTestVariant(test: SplitTest, variant = 1): boolean {
-  if (!hasConsentedTo('tracking')) return false
-
   return getSplitTestVariantIndex(test) === variant
 }
 

package/src/routes/components/Modals/TitlesReelModal.svelte

@@ -3,6 +3,8 @@
   import { onMount } from 'svelte'
   import { heading } from '$lib/actions/heading'
   import { t } from '$lib/localization'
+  import { track } from '$lib/tracking'
+  import { TrackingEvent } from '$lib/enums/TrackingEvent'
   import TinySlider from 'svelte-tiny-slider'
   import Modal from './Modal.svelte'
   import TitleBackground from '../TitleBackground.svelte'
@@ -15,9 +17,10 @@
 
   interface Props {
     titles: TitleData[],
+    onclose?: () => void
   }
 
-  const { titles }: Props = $props()
+  const { titles, onclose = () => null }: Props = $props()
 
   let interval: ReturnType<typeof setInterval> | null = null
 
@@ -29,6 +32,11 @@
     }
   })
 
+  function onchange(index: number): void {
+    pauseAndPlay(index)
+    track(TrackingEvent.ExploreTitleReelSetIndex, titles[index], { index })
+  }
+
   function pauseAndPlay(index: number): void {
     ;[titles[index - 1], titles[index + 1]].forEach(title => {
       window.dispatchEvent(new CustomEvent('pauseembed', { detail: title?.embeddable_url } ))
@@ -49,7 +57,7 @@
   <Modal>
     {#snippet dialog()}
       <div class="slider">
-        <TinySlider vertical threshold={50} moveThreshold={10} onchange={pauseAndPlay}>
+        <TinySlider vertical threshold={50} moveThreshold={10} {onchange}>
           {#snippet children({ currentIndex, setIndex })}
             {#each titles as title, index}
               <!-- svelte-ignore a11y_click_events_have_key_events -->
@@ -92,7 +100,7 @@
         </TinySlider>
       </div>
 
-      <ModalCloseOverlay label={t('Explore All Movies And Shows')} />
+      <ModalCloseOverlay {onclose} label={t('Explore All Movies And Shows')} />
     {/snippet}
   </Modal>
 </div>

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

@@ -2,11 +2,6 @@ import { describe, it, expect, vi, afterEach } from 'vitest'
 import { getSplitTestIdentifier, getSplitTestVariantIndex, getSplitTestVariantName, isInSplitTestVariant, sessionStoragePrefix, trackSplitTestAction, trackSplitTestView } from '$lib/splitTest'
 import { track } from '$lib/tracking'
 import { TrackingEvent } from '$lib/enums/TrackingEvent'
-import { hasConsentedTo } from '$lib/consent'
-
-vi.mock('$lib/consent', () => ({
-  hasConsentedTo: vi.fn(() => true),
-}))
 
 vi.mock('$lib/tracking', () => ({
   track: vi.fn(),
@@ -23,8 +18,6 @@ describe('$lib/splitTest', () => {
     window.PlayPilotLinkInjections = {}
 
     vi.resetAllMocks()
-
-    vi.mocked(hasConsentedTo).mockImplementation(() => true)
   })
 
   describe('getSplitTestIdentifier', () => {
@@ -60,12 +53,6 @@ describe('$lib/splitTest', () => {
       })
     })
 
-    it('Should return 0 if user did not consent', () => {
-      vi.mocked(hasConsentedTo).mockImplementation(() => false)
-
-      expect(getSplitTestIdentifier(splitTest)).toBe(0)
-    })
-
     it('Should store value in sessionStorage', () => {
       getSplitTestIdentifier(splitTest)
 
@@ -118,16 +105,6 @@ describe('$lib/splitTest', () => {
       expect(isInSplitTestVariant(multiple, 1)).toBe(false)
       expect(isInSplitTestVariant(multiple, 2)).toBe(true)
     })
-
-    it('Should return false if user did not consent', () => {
-      vi.mocked(hasConsentedTo).mockImplementation(() => false)
-
-      // @ts-ignore
-      window.PlayPilotLinkInjections.split_test_identifiers = {}
-      window.PlayPilotLinkInjections.split_test_identifiers[splitTest.key] = 0.75
-
-      expect(isInSplitTestVariant(splitTest)).toBe(false)
-    })
   })
 
   describe('trackSplitTestView', () => {