Fix an issue where 404s could cause a broken transition to nowhere

craftedbygc · Mar 20, 2024 · 60a9960 · 60a9960
1 parent 60823b0
commit 60a9960
Show file tree

Hide file tree

Showing 11 changed files with 22 additions and 17 deletions.
diff --git a/dist/taxi.esm.js b/dist/taxi.esm.js
diff --git a/dist/taxi.esm.js.map b/dist/taxi.esm.js.map
diff --git a/dist/taxi.js b/dist/taxi.js
diff --git a/dist/taxi.js.map b/dist/taxi.js.map
diff --git a/dist/taxi.modern.js b/dist/taxi.modern.js
diff --git a/dist/taxi.modern.js.map b/dist/taxi.modern.js.map
diff --git a/dist/taxi.umd.js b/dist/taxi.umd.js
diff --git a/dist/taxi.umd.js.map b/dist/taxi.umd.js.map
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,7 +1,7 @@
 {
 	"name": "@unseenco/taxi",
 	"description": "A modern page transition library which supports routing, preloading, and additional script reloading.",
-	"version": "1.5.1",
+	"version": "1.5.3",
 	"license": "GPL-3.0-or-later",
 	"source": "src/taxi.js",
 	"main": "src/taxi.js",

diff --git a/src/Core.js b/src/Core.js
@@ -144,6 +144,7 @@ export default class Core {
 						this.cache.get(url).renderer.createDom()
 					}
 				})
+				.catch(err => console.warn(err))
 		}
 
 		return Promise.resolve()
@@ -209,6 +210,10 @@ export default class Core {
 						this.cache.set(this.targetLocation.href, this.createCacheEntry(response.html, response.url))
 						this.cache.get(this.targetLocation.href).renderer.createDom()
 					})
+					.catch(err => {
+						// we encountered a 4** or 5** error, redirect to the requested URL
+						window.location.href = url
+					})
 
 				navigationPromise = this.beforeFetch(this.targetLocation, TransitionClass, trigger)
 					.then(async () => {