Fix the issue of the saved max shipping time not showing after revisi…

…ting the free listings editing page.
woocommerce · Dec 13, 2024 · 63ca753 · 63ca753
1 parent ff37b09
commit 63ca753
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 2 deletions.
diff --git a/js/src/data/reducer.js b/js/src/data/reducer.js
@@ -195,11 +195,11 @@ const reducer = ( state = DEFAULT_STATE, action ) => {
 		}
 
 		case TYPES.UPSERT_SHIPPING_TIMES: {
-			const { countryCodes, time } = action.shippingTime;
+			const { countryCodes, time, maxTime } = action.shippingTime;
 			const times = [ ...state.mc.shipping.times ];
 
 			countryCodes.forEach( ( countryCode ) => {
-				const shippingTime = { countryCode, time };
+				const shippingTime = { countryCode, time, maxTime };
 				const idx = times.findIndex(
 					( el ) => el.countryCode === countryCode
 				);

diff --git a/js/src/data/test/reducer.test.js b/js/src/data/test/reducer.test.js
@@ -261,10 +261,12 @@ describe( 'reducer', () => {
 					{
 						countryCode: 'US',
 						time: 7,
+						maxTime: 14,
 					},
 					{
 						countryCode: 'CA',
 						time: 12,
+						maxTime: 18,
 					},
 				],
 			};
@@ -278,17 +280,20 @@ describe( 'reducer', () => {
 				{
 					countryCode: 'US',
 					time: 7,
+					maxTime: 14,
 				},
 				{
 					countryCode: 'CA',
 					time: 12,
+					maxTime: 18,
 				},
 			] );
 			const action = {
 				type: TYPES.UPSERT_SHIPPING_TIMES,
 				shippingTime: {
 					countryCodes: [ 'JP', 'CA' ],
 					time: 15,
+					maxTime: 30,
 				},
 			};
 			const state = reducer( originalState, action );
@@ -297,14 +302,17 @@ describe( 'reducer', () => {
 				{
 					countryCode: 'US',
 					time: 7,
+					maxTime: 14,
 				},
 				{
 					countryCode: 'CA',
 					time: 15,
+					maxTime: 30,
 				},
 				{
 					countryCode: 'JP',
 					time: 15,
+					maxTime: 30,
 				},
 			] );
 		} );
@@ -314,14 +322,17 @@ describe( 'reducer', () => {
 				{
 					countryCode: 'US',
 					time: 7,
+					maxTime: 14,
 				},
 				{
 					countryCode: 'CA',
 					time: 12,
+					maxTime: 18,
 				},
 				{
 					countryCode: 'JP',
 					time: 15,
+					maxTime: 30,
 				},
 			] );
 			const action = {
@@ -334,6 +345,7 @@ describe( 'reducer', () => {
 				{
 					countryCode: 'CA',
 					time: 12,
+					maxTime: 18,
 				},
 			] );
 		} );