From 3e41d4de98bf3b0a58f4864843e933e847a6be00 Mon Sep 17 00:00:00 2001 From: Leah Schumann Date: Mon, 22 Aug 2022 16:29:29 -0400 Subject: [PATCH 1/2] Ensuring vin lookup page saveOrder is only triggered when we don't already have a session saved --- src/mixins/vin-pages-mixin.js | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/mixins/vin-pages-mixin.js b/src/mixins/vin-pages-mixin.js index 82be61557..eb6a7bb57 100644 --- a/src/mixins/vin-pages-mixin.js +++ b/src/mixins/vin-pages-mixin.js @@ -5,9 +5,11 @@ import { saveOrder } from "@/helpers/heritage-integration/order-helper.js"; export default { methods: { async navigateForwardWithSingleCarMatch() { - // We want to save the order before calling getPartsOrQuestions because that call is lengthy - saveOrder(); - + // If we have not already saved a session, we need to save one now before the lengthy call to getPartsOrQuestions + if (!store.getters.applicationUser.savedSessionId) { + saveOrder(); + } + const result = await this.dispatchStoreAction(storeActions.GET_PARTS_OR_QUESTIONS); const partsOrQuestions = result.data.partsOrQuestions; From 31ba6f361a7eb9b1f1628d81637dc0390eeec20c Mon Sep 17 00:00:00 2001 From: Leah Schumann Date: Mon, 22 Aug 2022 16:52:18 -0400 Subject: [PATCH 2/2] Added missed required reference to store --- src/mixins/vin-pages-mixin.js | 1 + src/mixins/vin-pages-mixin.spec.js | 7 ++++++- 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/src/mixins/vin-pages-mixin.js b/src/mixins/vin-pages-mixin.js index eb6a7bb57..ddda182ba 100644 --- a/src/mixins/vin-pages-mixin.js +++ b/src/mixins/vin-pages-mixin.js @@ -1,4 +1,5 @@ import { storeActions } from "@/constants/store-actions.js"; +import store from "@/store"; import vehicleQuestionsMixin from "@/mixins/vehicle-questions-mixin"; import { saveOrder } from "@/helpers/heritage-integration/order-helper.js"; diff --git a/src/mixins/vin-pages-mixin.spec.js b/src/mixins/vin-pages-mixin.spec.js index a4b2b136e..0ac493b42 100644 --- a/src/mixins/vin-pages-mixin.spec.js +++ b/src/mixins/vin-pages-mixin.spec.js @@ -50,7 +50,12 @@ function setupMocks({ partsOrQuestions = [] }) { navigate: jest.fn(), navigateWithSaving: jest.fn(), navigateWithoutSaving: jest.fn(), }, store: { - commit: jest.fn() + commit: jest.fn(), + getters: { + applicationUser: { + savedSessionId: 1 + } + } } });