From 5616dee3e2cb6da993e091002cad2a64899a13cd Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Fri, 24 Jul 2026 15:45:36 +0000 Subject: [PATCH 01/13] Bump shell-quote from 1.8.4 to 1.10.0 Bumps [shell-quote](https://github.com/ljharb/shell-quote) from 1.8.4 to 1.10.0. - [Changelog](https://github.com/ljharb/shell-quote/blob/main/CHANGELOG.md) - [Commits](https://github.com/ljharb/shell-quote/compare/v1.8.4...v1.10.0) --- updated-dependencies: - dependency-name: shell-quote dependency-version: 1.10.0 dependency-type: indirect ... Signed-off-by: dependabot[bot] --- package-lock.json | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/package-lock.json b/package-lock.json index f23f2e1a..a3ff0dca 100644 --- a/package-lock.json +++ b/package-lock.json @@ -18289,9 +18289,9 @@ } }, "node_modules/shell-quote": { - "version": "1.8.4", - "resolved": "https://registry.npmjs.org/shell-quote/-/shell-quote-1.8.4.tgz", - "integrity": "sha512-VsC6n6vz1ihYYyZZwX7YZSF5l5x36ca17OC+a69h94YqB7X6XLwf+5MOgynYir2SLFUbl8gIYvBo8K8RoNQ6bQ==", + "version": "1.10.0", + "resolved": "https://registry.npmjs.org/shell-quote/-/shell-quote-1.10.0.tgz", + "integrity": "sha512-w1aiOKwKuRgtwAReIIj89puqg+I7GvX4IbLrvmhXbzQsj1+Zwi4VO3+fa6ZF91TWSjIxoEkKnMeHcLEODK5ZXA==", "dev": true, "license": "MIT", "engines": { From 3bc87fd12803908122b3c12774c57b7e839e90f8 Mon Sep 17 00:00:00 2001 From: Alex Humphries Date: Fri, 24 Jul 2026 12:21:54 -0400 Subject: [PATCH 02/13] INSR-10489: Added state specific text to payment method page --- src/layouts/payment-method/payment-method.vue | 22 +++++++++++++++++-- 1 file changed, 20 insertions(+), 2 deletions(-) diff --git a/src/layouts/payment-method/payment-method.vue b/src/layouts/payment-method/payment-method.vue index d289df4f..02ad6851 100644 --- a/src/layouts/payment-method/payment-method.vue +++ b/src/layouts/payment-method/payment-method.vue @@ -19,6 +19,10 @@ +
Date: Fri, 24 Jul 2026 13:29:29 -0400 Subject: [PATCH 03/13] INSR-10489: Remove unneeded class, fix unneeded text, fix variable name --- src/layouts/payment-method/payment-method.vue | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/src/layouts/payment-method/payment-method.vue b/src/layouts/payment-method/payment-method.vue index 02ad6851..2d6e5237 100644 --- a/src/layouts/payment-method/payment-method.vue +++ b/src/layouts/payment-method/payment-method.vue @@ -21,7 +21,6 @@ cmsWidgetName="SiteSubHeaderWidget" />
@@ -324,9 +323,9 @@ export default { }; }, stateSpecificText() { - const stateSpecificText = this.getCmsContent(this.widget.stateSpecificText, widgetFields.TEXT_BLOCK_WIDGET.TEXT); - if (stateSpecificText) { - return processIfStatements(stateSpecificText, 'custom', this.getStateSpecificText); + const rawStateSpecificText = this.getCmsContent(this.widget.stateSpecificText, widgetFields.TEXT_BLOCK_WIDGET.TEXT); + if (rawStateSpecificText) { + return processIfStatements(rawStateSpecificText, 'custom', this.getStateSpecificText); } return null; } @@ -578,7 +577,6 @@ export default { action = 'Unverified'; label = 'Unverified'; break; - d } this.pushEventToGA('Submit Appointment', action, label, true, null, 1); }, From 3edc040b6ab9279a84cc4f70490536fb5045d1c0 Mon Sep 17 00:00:00 2001 From: Alex Humphries Date: Fri, 24 Jul 2026 15:07:14 -0400 Subject: [PATCH 04/13] INSR-10489: Add benefit-recal-modal to schedule-page --- .../benefit-recal-modal.vue | 155 ++++++++++++++++++ src/layouts/schedule-page/schedule-page.vue | 42 ++++- 2 files changed, 194 insertions(+), 3 deletions(-) create mode 100644 src/layouts/schedule-page/benefit-recal-modal/benefit-recal-modal.vue diff --git a/src/layouts/schedule-page/benefit-recal-modal/benefit-recal-modal.vue b/src/layouts/schedule-page/benefit-recal-modal/benefit-recal-modal.vue new file mode 100644 index 00000000..65092705 --- /dev/null +++ b/src/layouts/schedule-page/benefit-recal-modal/benefit-recal-modal.vue @@ -0,0 +1,155 @@ + + + + + \ No newline at end of file diff --git a/src/layouts/schedule-page/schedule-page.vue b/src/layouts/schedule-page/schedule-page.vue index 6c3af64c..0cb37f94 100644 --- a/src/layouts/schedule-page/schedule-page.vue +++ b/src/layouts/schedule-page/schedule-page.vue @@ -55,7 +55,7 @@ cmsWidgetName="SiteFooterWidget" :isForwardButtonHidden="!hasNeededServiceLocationData" :isForwardButtonNavigationDisabled="!isFormValid" - @backClicked="navigateBack(this, navigateBackScenario)" + @backClicked="backButtonAction" @forwardClicked="forwardButtonAction" />
@@ -69,6 +69,11 @@ cmsWidgetName="RecalAckModalWidget" :ackError="ackError" @recalAcknowledged="updateIsRecalAcknowledged"/> + From b5620e087e712640f97b607e043c6b7590f84610 Mon Sep 17 00:00:00 2001 From: Alex Humphries Date: Fri, 24 Jul 2026 22:25:26 -0400 Subject: [PATCH 05/13] INSR-10489: Remove debug logging --- .../schedule-page/benefit-recal-modal/benefit-recal-modal.vue | 2 -- src/layouts/schedule-page/schedule-page.vue | 2 -- 2 files changed, 4 deletions(-) diff --git a/src/layouts/schedule-page/benefit-recal-modal/benefit-recal-modal.vue b/src/layouts/schedule-page/benefit-recal-modal/benefit-recal-modal.vue index 65092705..a567c2a2 100644 --- a/src/layouts/schedule-page/benefit-recal-modal/benefit-recal-modal.vue +++ b/src/layouts/schedule-page/benefit-recal-modal/benefit-recal-modal.vue @@ -87,11 +87,9 @@ export default { onModalClosed() { this.pushEvent(); if (!this.schedulingElsewhere) { - console.log('Emitting continue-safelite-clicked event'); this.$emit('continue-safelite-clicked'); } else { - console.log('Emitting schedule-elsewhere-clicked event'); this.$emit('schedule-elsewhere-clicked'); } }, diff --git a/src/layouts/schedule-page/schedule-page.vue b/src/layouts/schedule-page/schedule-page.vue index 0cb37f94..1d4b9c85 100644 --- a/src/layouts/schedule-page/schedule-page.vue +++ b/src/layouts/schedule-page/schedule-page.vue @@ -1031,11 +1031,9 @@ export default { return this.mainStore.order.isRecalAcknowledgedForScheduling; }, continueWithSafeliteClicked() { - console.log('continueWithSafeliteClicked'); this.benefitRecalModalAcknowledged = true; }, scheduleElsewhereClicked() { - console.log('scheduleElsewhereClicked'); this.navigateBack(this, this.navigateBackScenario) }, backButtonAction() { From ca8d77d2c26527c85c9eb03c3ac564c83aee80fe Mon Sep 17 00:00:00 2001 From: Alex Humphries Date: Fri, 24 Jul 2026 22:26:57 -0400 Subject: [PATCH 06/13] INSR-10489: Fix inconsistent indentation and missing semicolons --- src/layouts/schedule-page/schedule-page.vue | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/src/layouts/schedule-page/schedule-page.vue b/src/layouts/schedule-page/schedule-page.vue index 1d4b9c85..b42de065 100644 --- a/src/layouts/schedule-page/schedule-page.vue +++ b/src/layouts/schedule-page/schedule-page.vue @@ -1034,21 +1034,21 @@ export default { this.benefitRecalModalAcknowledged = true; }, scheduleElsewhereClicked() { - this.navigateBack(this, this.navigateBackScenario) + this.navigateBack(this, this.navigateBackScenario); }, backButtonAction() { if (this.hasBenefitRecalModal && !this.benefitRecalModalAcknowledged) { this.$refs[BENEFIT_RECAL_MODAL_REF_NAME].openModal(); const gaLabel = this.mainStore.hasRecalibrationPart ? 'recalibration_version' : 'non-recalibration_version'; - this.pushEventToGA( - 'safelite_benefit_modal_displayed', - this.mainStore.accountNameForEvents, - gaLabel, - true - ); + this.pushEventToGA( + 'safelite_benefit_modal_displayed', + this.mainStore.accountNameForEvents, + gaLabel, + true + ); } else { - this.navigateBack(this, this.navigateBackScenario) + this.navigateBack(this, this.navigateBackScenario); } } } From 1f7e5f10ecd483f7f5a117b251da66b16e64ba32 Mon Sep 17 00:00:00 2001 From: Alex Humphries Date: Fri, 24 Jul 2026 23:02:28 -0400 Subject: [PATCH 07/13] INSR-10489: Add tests related to benefit recal modal to schedule page --- .../schedule-page/schedule-page.spec.js | 115 +++++++++++++++++- 1 file changed, 113 insertions(+), 2 deletions(-) diff --git a/src/layouts/schedule-page/schedule-page.spec.js b/src/layouts/schedule-page/schedule-page.spec.js index 411f7e4f..00829a6a 100644 --- a/src/layouts/schedule-page/schedule-page.spec.js +++ b/src/layouts/schedule-page/schedule-page.spec.js @@ -13,6 +13,7 @@ jest.mock('@/helpers/cms-content-helper', () => ({ })); const RECAL_ACK_MODAL_REF_NAME = 'recalAckModal'; +const BENEFIT_RECAL_MODAL_REF_NAME = 'benefitRecalModal'; const mockMixin = { methods: { @@ -74,6 +75,18 @@ const recalAckModalStub = { } }; +const benefitRecalModalStub = { + template: '
', + methods: { + openModal: jest.fn(), + footerButtonClick: jest.fn() + } +}; + +const mockBenefitRecalModalContent = { + HeaderText: 'Benefit Recalibration Modal Header' +} + function getShallowMountedComponent(initialData = {}, methodToRun = () => {}) { const mountOptions = getMountOptions({ router: { @@ -85,7 +98,8 @@ function getShallowMountedComponent(initialData = {}, methodToRun = () => {}) { siteFooter: footerStub, loadingModal: loadingModalStub, serviceLocation: serviceLocationStub, - recalAckModal: recalAckModalStub + recalAckModal: recalAckModalStub, + benefitRecalModal: benefitRecalModalStub }; methodToRun(); @@ -106,6 +120,16 @@ function getShallowMountedComponent(initialData = {}, methodToRun = () => {}) { writable: false, configurable: true }); + + // Manually create the ref for benefitRecalModal since shallowMount doesn't populate it + Object.defineProperty(wrapper.vm.$refs, BENEFIT_RECAL_MODAL_REF_NAME, { + value: { + openModal: jest.fn(), + footerButtonClick: jest.fn() + }, + writable: false, + configurable: true + }); // Manually create the ref for serviceLocation since shallowMount doesn't populate it Object.defineProperty(wrapper.vm.$refs, 'serviceLocation', { @@ -158,6 +182,9 @@ beforeEach(() => { lineItems: { glassParts: [], supportingItems: [] + }, + issConfig: { + clientName: 'TestClient' } } } @@ -390,4 +417,88 @@ describe('schedule page methods...', () => { // Assert expect(wrapper.vm.$router.navigate).toHaveBeenCalled(); }); -}); + test('backButtonAction opens benefitRecalModal when hasBenefitRecalModal returns true and benefitRecalModalAcknowledged is false', () => { + // Arrange + const { wrapper } = getShallowMountedComponent(); + wrapper.vm.getCmsContent = jest.fn().mockReturnValue('Benefit Recalibration Modal Header'); + wrapper.vm.benefitRecalModalAcknowledged = false; + + // Act + wrapper.vm.backButtonAction(); + + // Assert + expect(wrapper.vm.$refs[BENEFIT_RECAL_MODAL_REF_NAME].openModal).toHaveBeenCalled(); + }); + test('backButtonAction pushes non-recalibration_version event when opening benefitRecalModal and hasRecalibrationPart is false', () => { + // Arrange + const { wrapper } = getShallowMountedComponent(); + wrapper.vm.getCmsContent = jest.fn().mockReturnValue('Benefit Recalibration Modal Header'); + wrapper.vm.benefitRecalModalAcknowledged = false; + wrapper.vm.mainStore.lineItems.glassParts = [ + { + partNumber: 'TEST123', + requiresRecalibration: false + } + ]; + wrapper.vm.pushEventToGA = jest.fn(); + + // Act + wrapper.vm.backButtonAction(); + + // Assert + expect(wrapper.vm.pushEventToGA).toHaveBeenCalledWith( + 'safelite_benefit_modal_displayed', + wrapper.vm.mainStore.accountNameForEvents, + 'non-recalibration_version', + true + ); + }); + test('backButtonAction pushes recalibration_version event when opening benefitRecalModal and hasRecalibrationPart is true', () => { + // Arrange + const { wrapper } = getShallowMountedComponent(); + wrapper.vm.getCmsContent = jest.fn().mockReturnValue('Benefit Recalibration Modal Header'); + wrapper.vm.benefitRecalModalAcknowledged = false; + wrapper.vm.mainStore.lineItems.glassParts = [ + { + partNumber: 'TEST123', + requiresRecalibration: true + } + ]; + wrapper.vm.pushEventToGA = jest.fn(); + + // Act + wrapper.vm.backButtonAction(); + + // Assert + expect(wrapper.vm.pushEventToGA).toHaveBeenCalledWith( + 'safelite_benefit_modal_displayed', + wrapper.vm.mainStore.accountNameForEvents, + 'recalibration_version', + true + ); + }); + test('backButtonAction navigates back when hasBenefitRecalModal is true and benefitRecalModalAcknowledged is true', () => { + // Arrange + const { wrapper } = getShallowMountedComponent(); + wrapper.vm.getCmsContent = jest.fn().mockReturnValue('Benefit Recalibration Modal Header'); + wrapper.vm.benefitRecalModalAcknowledged = true; + wrapper.vm.navigateBack = jest.fn(); + + // Act + wrapper.vm.backButtonAction(); + + // Assert + expect(wrapper.vm.navigateBack).toHaveBeenCalled(); + }); + test('backButtonAction navigates back when hasBenefitRecalModal is false', () => { + // Arrange + const { wrapper } = getShallowMountedComponent(); + wrapper.vm.navigateBack = jest.fn(); + + // Act + wrapper.vm.backButtonAction(); + + // Assert + expect(wrapper.vm.navigateBack).toHaveBeenCalled(); + }); +}); \ No newline at end of file From 79894f18f7d9e2342d3e04dc7e191c07624cd1eb Mon Sep 17 00:00:00 2001 From: Alex Humphries Date: Fri, 24 Jul 2026 23:14:29 -0400 Subject: [PATCH 08/13] INSR-10489: Add tests for state-specific text on payment-method --- .../payment-method/payment-method.spec.js | 30 +++++++++++++++++-- src/layouts/payment-method/payment-method.vue | 1 + 2 files changed, 29 insertions(+), 2 deletions(-) diff --git a/src/layouts/payment-method/payment-method.spec.js b/src/layouts/payment-method/payment-method.spec.js index edee79f9..74051c98 100644 --- a/src/layouts/payment-method/payment-method.spec.js +++ b/src/layouts/payment-method/payment-method.spec.js @@ -20,7 +20,7 @@ jest.mock('@/helpers/order-helper.js', () => ({ saveSession: jest.fn() })); -function setupMocks({ customMountOptions = {}, queryString }, mainInitialState = {}, customMixin = null) { +function setupMocks({ customMountOptions = {}, queryString }, mainInitialState = {}, customMixin = null, mockCmsContent = '') { const mountOptions = getMountOptions({ ...customMountOptions, route: { query: { issPage: issPageValues.PAYMENT_METHOD, ...queryString }, params: {} }, @@ -51,7 +51,9 @@ function setupMocks({ customMountOptions = {}, queryString }, mainInitialState = }; const cmsMixin = { methods: { - getCmsContent: jest.fn() + getCmsContent: jest.fn((widget, field) => { + return mockCmsContent; + }) } } @@ -320,4 +322,28 @@ describe('payment-method.vue', () => { expect(submitWorkOrder).toHaveBeenCalledTimes(1); }); }); + + describe('State Specific Text Rendering', () => { + test('State specific textBlock component exists when CMS content for it exists', () => { + // Arrange + const mockCmsContent = 'This is a test CMS content for state specific text.'; + const wrapper = setupMocks({}, {}, null, mockCmsContent); + + // Act + const stateSpecificTextBlock = wrapper.find('[class="state-specific-text"]'); + + // Assert + expect(stateSpecificTextBlock.exists()).toBe(true); + }); + test('State specific textBlock component does not exist when CMS content for it does not exist', () => { + // Arrange + const wrapper = setupMocks({}, {}, null, ''); + + // Act + const stateSpecificTextBlock = wrapper.find('[class="state-specific-text"]'); + + // Assert + expect(stateSpecificTextBlock.exists()).toBe(false); + }); + }); }); diff --git a/src/layouts/payment-method/payment-method.vue b/src/layouts/payment-method/payment-method.vue index 2d6e5237..95b097a8 100644 --- a/src/layouts/payment-method/payment-method.vue +++ b/src/layouts/payment-method/payment-method.vue @@ -21,6 +21,7 @@ cmsWidgetName="SiteSubHeaderWidget" />
From d63cca3b8f146c68b7342f71448beb8b210d1551 Mon Sep 17 00:00:00 2001 From: AHumphriesSL Date: Fri, 24 Jul 2026 23:20:31 -0400 Subject: [PATCH 09/13] Remove unneeded mock Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --- src/layouts/schedule-page/schedule-page.spec.js | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/layouts/schedule-page/schedule-page.spec.js b/src/layouts/schedule-page/schedule-page.spec.js index 00829a6a..efbc7bdd 100644 --- a/src/layouts/schedule-page/schedule-page.spec.js +++ b/src/layouts/schedule-page/schedule-page.spec.js @@ -83,9 +83,6 @@ const benefitRecalModalStub = { } }; -const mockBenefitRecalModalContent = { - HeaderText: 'Benefit Recalibration Modal Header' -} function getShallowMountedComponent(initialData = {}, methodToRun = () => {}) { const mountOptions = getMountOptions({ From 5686afa931405933bfb3f04e21e64291cc9b3bce Mon Sep 17 00:00:00 2001 From: Alex Humphries Date: Fri, 24 Jul 2026 23:26:48 -0400 Subject: [PATCH 10/13] INSR-10489: Fix mock store duplicating lineItems on schedule=page.spec --- src/layouts/schedule-page/schedule-page.spec.js | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/src/layouts/schedule-page/schedule-page.spec.js b/src/layouts/schedule-page/schedule-page.spec.js index efbc7bdd..22e96af9 100644 --- a/src/layouts/schedule-page/schedule-page.spec.js +++ b/src/layouts/schedule-page/schedule-page.spec.js @@ -176,10 +176,6 @@ beforeEach(() => { payment: { isInsurance: true }, - lineItems: { - glassParts: [], - supportingItems: [] - }, issConfig: { clientName: 'TestClient' } @@ -364,7 +360,7 @@ describe('schedule page methods...', () => { test('recalAckModal should open when part requires recal, but recal not added to order', async () => { // Arrange const { wrapper } = getShallowMountedComponent(); - wrapper.vm.mainStore.lineItems.glassParts = [ + wrapper.vm.mainStore.order.lineItems.glassParts = [ { partNumber: 'TEST123', requiresRecalibration: true @@ -380,7 +376,7 @@ describe('schedule page methods...', () => { test('navigation forward is blocked when recalAckModal is displayed, but checkbox has not been acknowledged', async () => { // Arrange const { wrapper } = getShallowMountedComponent(); - wrapper.vm.mainStore.lineItems.glassParts = [ + wrapper.vm.mainStore.order.lineItems.glassParts = [ { partNumber: 'TEST123', requiresRecalibration: true @@ -398,7 +394,7 @@ describe('schedule page methods...', () => { test('navigation forward is allowed when checkbox has been acknowledged on recalAckModal', async () => { // Arrange const { wrapper } = getShallowMountedComponent(); - wrapper.vm.mainStore.lineItems.glassParts = [ + wrapper.vm.mainStore.order.lineItems.glassParts = [ { partNumber: 'TEST123', requiresRecalibration: true @@ -431,7 +427,7 @@ describe('schedule page methods...', () => { const { wrapper } = getShallowMountedComponent(); wrapper.vm.getCmsContent = jest.fn().mockReturnValue('Benefit Recalibration Modal Header'); wrapper.vm.benefitRecalModalAcknowledged = false; - wrapper.vm.mainStore.lineItems.glassParts = [ + wrapper.vm.mainStore.order.lineItems.glassParts = [ { partNumber: 'TEST123', requiresRecalibration: false @@ -455,7 +451,7 @@ describe('schedule page methods...', () => { const { wrapper } = getShallowMountedComponent(); wrapper.vm.getCmsContent = jest.fn().mockReturnValue('Benefit Recalibration Modal Header'); wrapper.vm.benefitRecalModalAcknowledged = false; - wrapper.vm.mainStore.lineItems.glassParts = [ + wrapper.vm.mainStore.order.lineItems.glassParts = [ { partNumber: 'TEST123', requiresRecalibration: true From 413391893fa3e7a5bf53fafa4051d1fd8523de47 Mon Sep 17 00:00:00 2001 From: Alex Humphries Date: Fri, 24 Jul 2026 23:42:06 -0400 Subject: [PATCH 11/13] INSR-10489: A bunch of formatting fixes --- .../payment-method/payment-method.spec.js | 4 ++-- src/layouts/payment-method/payment-method.vue | 4 ++-- .../benefit-recal-modal/benefit-recal-modal.vue | 16 ++++++++-------- 3 files changed, 12 insertions(+), 12 deletions(-) diff --git a/src/layouts/payment-method/payment-method.spec.js b/src/layouts/payment-method/payment-method.spec.js index 74051c98..a70be241 100644 --- a/src/layouts/payment-method/payment-method.spec.js +++ b/src/layouts/payment-method/payment-method.spec.js @@ -330,7 +330,7 @@ describe('payment-method.vue', () => { const wrapper = setupMocks({}, {}, null, mockCmsContent); // Act - const stateSpecificTextBlock = wrapper.find('[class="state-specific-text"]'); + const stateSpecificTextBlock = wrapper.find('.state-specific-text'); // Assert expect(stateSpecificTextBlock.exists()).toBe(true); @@ -340,7 +340,7 @@ describe('payment-method.vue', () => { const wrapper = setupMocks({}, {}, null, ''); // Act - const stateSpecificTextBlock = wrapper.find('[class="state-specific-text"]'); + const stateSpecificTextBlock = wrapper.find('.state-specific-text'); // Assert expect(stateSpecificTextBlock.exists()).toBe(false); diff --git a/src/layouts/payment-method/payment-method.vue b/src/layouts/payment-method/payment-method.vue index 95b097a8..5c87754a 100644 --- a/src/layouts/payment-method/payment-method.vue +++ b/src/layouts/payment-method/payment-method.vue @@ -582,8 +582,8 @@ export default { this.pushEventToGA('Submit Appointment', action, label, true, null, 1); }, getStateSpecificText(value) { - if(value.includes('not')) { - return !value?.toLowerCase().includes(this.mainStore.order.customer.address.state?.toLowerCase()) + if (value.includes('not')) { + return !value?.toLowerCase().includes(this.mainStore.order.customer.address.state?.toLowerCase()); } return this.mainStore.order.customer.address.state?.toLowerCase() === value?.toLowerCase(); } diff --git a/src/layouts/schedule-page/benefit-recal-modal/benefit-recal-modal.vue b/src/layouts/schedule-page/benefit-recal-modal/benefit-recal-modal.vue index a567c2a2..6c287a0e 100644 --- a/src/layouts/schedule-page/benefit-recal-modal/benefit-recal-modal.vue +++ b/src/layouts/schedule-page/benefit-recal-modal/benefit-recal-modal.vue @@ -61,7 +61,7 @@ export default { }, computed: { modalBodyText() { - return this.getCmsContent(this.cmsWidgetName, widgetFields.CONTENT_GROUP_WIDGET.BODY_TEXT) + return this.getCmsContent(this.cmsWidgetName, widgetFields.CONTENT_GROUP_WIDGET.BODY_TEXT); }, modalCloseLinkText() { return this.getCmsContent(this.cmsWidgetName, widgetFields.CONTENT_GROUP_WIDGET.BODY_TEXT_2); @@ -98,12 +98,12 @@ export default { }, pushEvent() { const gaLabel = this.schedulingElsewhere ? 'schedule_elsewhere_clicked' : 'continue_with_safelite_clicked'; - this.pushEventToGA( - 'safelite_benefit_modal', - 'modal_closed', - gaLabel, - true - ); + this.pushEventToGA( + 'safelite_benefit_modal', + 'modal_closed', + gaLabel, + true + ); } } }; @@ -128,7 +128,7 @@ export default { background-color: $green; color: $white; border-color: $blue; - font-weight: $font-weight-bold; + font-weight: $font-weight-bold; &:hover { background-color: $green; color: $white; From ceddfee3adbfd68bb0b1a8a341e9f85965a89b64 Mon Sep 17 00:00:00 2001 From: AHumphriesSL Date: Fri, 24 Jul 2026 23:50:09 -0400 Subject: [PATCH 12/13] Remove redundant props Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --- .../schedule-page/benefit-recal-modal/benefit-recal-modal.vue | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/layouts/schedule-page/benefit-recal-modal/benefit-recal-modal.vue b/src/layouts/schedule-page/benefit-recal-modal/benefit-recal-modal.vue index 6c287a0e..4608f780 100644 --- a/src/layouts/schedule-page/benefit-recal-modal/benefit-recal-modal.vue +++ b/src/layouts/schedule-page/benefit-recal-modal/benefit-recal-modal.vue @@ -5,8 +5,6 @@ ref="benefitRecalModal" :displayCloseButton="true" :displayFooterButton="false" - :closeLinkText="modalCloseLinkText" - :footerButtonText="modalFooterButtonText" :onModalClosedCallback="onModalClosed"> Date: Fri, 24 Jul 2026 23:55:50 -0400 Subject: [PATCH 13/13] Make hasBenefitRecalModal return a boolean value Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --- src/layouts/schedule-page/schedule-page.vue | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/layouts/schedule-page/schedule-page.vue b/src/layouts/schedule-page/schedule-page.vue index b42de065..43b17ba0 100644 --- a/src/layouts/schedule-page/schedule-page.vue +++ b/src/layouts/schedule-page/schedule-page.vue @@ -451,7 +451,7 @@ export default { return errorMessages.ACKNOWLEDGEMENT_REQUIRED; }, hasBenefitRecalModal() { - return this.getCmsContent('BenefitRecalModalWidget', 'HeaderText'); + return !!this.getCmsContent('BenefitRecalModalWidget', 'HeaderText'); } }, watch: {