@playpilot/tpi 6.2.1 → 6.2.3

package/package.json

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

package/src/lib/api/region.ts

@@ -10,12 +10,14 @@ export async function getRegionBasedOnIp(timeout = 5000): Promise<string> {
 
     const data: { ip: string, country: string } = await response.json()
 
-    const country = data?.country?.toLowerCase()
-    if (!country) throw new Error('No country was returned by api.country.is')
+    let region = data?.country?.toLowerCase()
+    if (!region) throw new Error('No country was returned by api.country.is')
 
-    setSavedRegion(country)
+    if (region === 'gb') region = 'uk'
 
-    return country
+    setSavedRegion(region)
+
+    return region
   } catch (error: any) {
     track(TrackingEvent.RegionRequestFailed, null, { message: error.message })
 

package/src/routes/+page.svelte

@@ -72,12 +72,12 @@
 
       window.PlayPilotLinkInjections.config = config
 
-      isUrlExcluded = !!(config?.exclude_urls_pattern && url.match(new RegExp(config.exclude_urls_pattern)))
-      if (isUrlExcluded) return
-
       if (config?.custom_style) insertCustomStyle(config.custom_style || '')
       if (config?.explore_navigation_selector) insertExploreIntoNavigation()
 
+      isUrlExcluded = !!(config?.exclude_urls_pattern && url.match(new RegExp(config.exclude_urls_pattern)))
+      if (isUrlExcluded) return
+
       setElements(config?.html_selector || '', config?.exclude_elements_selector || '')
     } catch(error) {
       // We return if the config did not get fetched properly, as we can't determine what should and should

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

@@ -87,4 +87,10 @@ describe('getRegionBasedOnIp', () => {
 
     vi.useRealTimers()
   })
+
+  it('Should use uk region when GB is returned by api endpoint', async () => {
+    fakeFetch({ response: { ip: '127.0.0.1', country: 'GB' } })
+
+    expect(await getRegionBasedOnIp()).toBe('uk')
+  })
 })