fix: move ftd settings query to checkoutSettings file, add basic sent…

…ry log for missing data
kiva · Nov 6, 2023 · 646915b · 646915b
1 parent 92c3877
commit 646915b
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 12 deletions.
diff --git a/src/graphql/query/checkout/checkoutSettings.graphql b/src/graphql/query/checkout/checkoutSettings.graphql
@@ -4,6 +4,18 @@ query checkoutSettings($basketId: String) {
 			value
 			key
 		}
+		ftd_message_enable: uiConfigSetting(key: "ftd_message_enable") {
+			key
+			value
+		}
+		ftd_message_amount: uiConfigSetting(key: "ftd_message_amount") {
+			key
+			value
+		}
+		ftd_message_valid_date: uiConfigSetting(key: "ftd_message_valid_date") {
+			key
+			value
+		}
 	}
 	shop (basketId: $basketId) {
 		id

diff --git a/src/graphql/query/checkout/initializeCheckout.graphql b/src/graphql/query/checkout/initializeCheckout.graphql
@@ -34,18 +34,6 @@ query initializeCheckout($basketId: String) {
 			value
 			description
 		}
-		ftd_message_enable: uiConfigSetting(key: "ftd_message_enable") {
-			key
-			value
-		}
-		ftd_message_amount: uiConfigSetting(key: "ftd_message_amount") {
-			key
-			value
-		}
-		ftd_message_valid_date: uiConfigSetting(key: "ftd_message_valid_date") {
-			key
-			value
-		}
 	}
 	shop (basketId: $basketId) {
 		id

diff --git a/src/pages/Checkout/CheckoutPage.vue b/src/pages/Checkout/CheckoutPage.vue
@@ -523,6 +523,14 @@ export default {
 		const validationErrors = _get(shopMutationData, 'validatePreCheckout', []);
 		this.showCheckoutError(validationErrors, true);
 
+		// Fire error for empty shop state
+		if (!this.isServer && !this.totals) {
+			Sentry.withScope(scope => {
+				scope.setTag('init_checkout', 'Missing baseline basket information');
+				Sentry.captureMessage(`Missing baseline basket information; totals value is ${this.totals}.`);
+			});
+		}
+
 		// TODO: Implement check against contentful setting
 		// to signify if holiday mode is enabled