Save WIP.

woocommerce · Oct 31, 2024 · 3c19aa9 · 3c19aa9
1 parent 9f335dd
commit 3c19aa9
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 9 deletions.
diff --git a/js/src/components/google-combo-account-card/claim-ads-account/claim-ads-account.js b/js/src/components/google-combo-account-card/claim-ads-account/claim-ads-account.js
@@ -60,10 +60,6 @@ const ClaimAdsAccount = () => {
 		setUpdating( true );
 	};
 
-	if ( ! shouldClaimGoogleAdsAccount ) {
-		return null;
-	}
-
 	return (
 		<Section.Card.Body className="gla-claim-ads-account-section">
 			<div className="gla-claim-ads-account-box">

diff --git a/js/src/components/google-combo-account-card/connect-ads/connect-ads.js b/js/src/components/google-combo-account-card/connect-ads/connect-ads.js
@@ -57,6 +57,7 @@ const ConnectAds = () => {
 		await upsertAdsAccount();
 	};
 
+	console.log( accounts, googleAdsAccount, isConnected );
 	useEffect( () => {
 		if ( isConnected ) {
 			setValue( googleAdsAccount.id );

diff --git a/js/src/components/google-combo-account-card/connected-accounts-actions.js b/js/src/components/google-combo-account-card/connected-accounts-actions.js
@@ -12,14 +12,16 @@ import useGoogleAdsAccountReady from '.~/hooks/useGoogleAdsAccountReady';
 
 /**
  * Renders the actionable components for connected accounts.
+ * @param {Object} props Component props.
+ * @param {boolean} props.claimGoogleAdsAccount Whether the user should claim the Google Ads account.
  * @return {JSX.Element} Connected accounts actions.
  */
-const ConnectedAccountsActions = () => {
+const ConnectedAccountsActions = ( { claimGoogleAdsAccount } ) => {
 	const isReady = useGoogleAdsAccountReady();
 
 	return (
 		<div className="gla-connected-accounts-actions">
-			<ClaimAdsAccount />
+			{ claimGoogleAdsAccount && <ClaimAdsAccount /> }
 
 			{ isReady && (
 				<AppNotice

diff --git a/js/src/components/google-combo-account-card/connected-google-combo-account-card.js b/js/src/components/google-combo-account-card/connected-google-combo-account-card.js
@@ -39,8 +39,8 @@ const ConnectedGoogleComboAccountCard = () => {
 	);
 	const hasExistingGoogleAdsAccounts = existingGoogleAdsAccounts?.length > 0;
 	const showConnectAds =
-		( editMode && hasExistingGoogleAdsAccounts ) ||
-		( ! isConnected && hasExistingGoogleAdsAccounts ) ||
+		( ( editMode && hasExistingGoogleAdsAccounts ) ||
+			( ! isConnected && hasExistingGoogleAdsAccounts ) ) &&
 		! shouldClaimGoogleAdsAccount;
 
 	return (
@@ -55,7 +55,9 @@ const ConnectedGoogleComboAccountCard = () => {
 					<Indicator showSpinner={ Boolean( creatingWhich ) } />
 				}
 			>
-				<ConnectedAccountsActions />
+				<ConnectedAccountsActions
+					claimGoogleAdsAccount={ shouldClaimGoogleAdsAccount }
+				/>
 			</AccountCard>
 
 			{ showConnectAds && <ConnectAds /> }

diff --git a/js/src/hooks/useAutoCreateAdsMCAccounts.js b/js/src/hooks/useAutoCreateAdsMCAccounts.js
@@ -28,6 +28,7 @@ const useShouldCreateAdsAccount = () => {
 		hasFinishedResolution: hasResolvedExistingAccounts,
 		existingAccounts: accounts,
 	} = useExistingGoogleAdsAccounts();
+	console.log( accounts, hasConnection );
 
 	// Return null if the account hasn't been resolved or the existing accounts haven't been resolved
 	if ( ! hasResolvedAccount || ! hasResolvedExistingAccounts ) {