Merge pull request #32926 from woocommerce/fix/32896_two_tasklists

Fix duplicate task list shown in some occurrences

packages/js/data/CHANGELOG.md

@@ -13,6 +13,7 @@
 -   Convert `use-user-preferences.js` to TS. #32695
 -   Added PaymentGateway type to exports #32697
 -   Add `@types/wordpress__compose`, `@types/wordpress__data`, `redux` types and fix related type errors. #32735
+-   Fix issue in `onboarding` data package for the unhide and hide success actions. #32926
 
 ## Breaking change
 

packages/js/data/src/onboarding/actions.js

@@ -169,6 +169,7 @@ export function hideTaskListSuccess( taskList ) {
 	return {
 		type: TYPES.HIDE_TASK_LIST_SUCCESS,
 		taskList,
+		taskListId: taskList.id,
 	};
 }
 
@@ -191,6 +192,7 @@ export function unhideTaskListSuccess( taskList ) {
 	return {
 		type: TYPES.UNHIDE_TASK_LIST_SUCCESS,
 		taskList,
+		taskListId: taskList.id,
 	};
 }
 

plugins/woocommerce-admin/client/activity-panel/activity-panel.js

@@ -246,7 +246,7 @@ export const ActivityPanel = ( { isEmbedded, query } ) => {
 				if ( currentLocation !== homescreenLocation ) {
 					// Ensure that if the user is trying to get to the task list they can see it even if
 					// it was dismissed.
-					if ( setupTaskListHidden === 'no' ) {
+					if ( setupTaskListHidden === false ) {
 						redirectToHomeScreen();
 					} else {
 						unhideTaskList( 'setup' ).then( redirectToHomeScreen );

plugins/woocommerce/changelog/fix-32896_two_tasklists

@@ -0,0 +1,4 @@
+Significance: patch
+Type: fix
+
+Update finish setup button logic to adhere to new Task List data structure. #32926