From a70cf525f98aaab0e69259e4147eb54ae185bc24 Mon Sep 17 00:00:00 2001
From: Adam Cassis <adam.cassis@automattic.com>
Date: Thu, 19 Sep 2024 13:22:31 +0200
Subject: [PATCH] feat: update UI

---
 .../class-newspack-ads-configuration-manager.php    |  4 +++-
 src/components/src/action-card/index.js             |  3 ++-
 src/wizards/advertising/components/add-ons/index.js | 13 +++++++++----
 3 files changed, 14 insertions(+), 6 deletions(-)

diff --git a/includes/configuration_managers/class-newspack-ads-configuration-manager.php b/includes/configuration_managers/class-newspack-ads-configuration-manager.php
index a99da83975..c1ad4456b4 100644
--- a/includes/configuration_managers/class-newspack-ads-configuration-manager.php
+++ b/includes/configuration_managers/class-newspack-ads-configuration-manager.php
@@ -207,7 +207,9 @@ public static function get_media_kit_page_id() {
 	 * Get Media Kit page.
 	 */
 	public function get_media_kit_page_edit_url() {
-		return get_edit_post_link( $this->get_media_kit_page_id(), '' );
+		$raw_url = get_edit_post_link( $this->get_media_kit_page_id(), '' );
+		$parsed_url = wp_parse_url( $raw_url );
+		return $parsed_url['path'] . '?' . $parsed_url['query'];
 	}
 
 	/**
diff --git a/src/components/src/action-card/index.js b/src/components/src/action-card/index.js
index 81188327ee..781a64164d 100644
--- a/src/components/src/action-card/index.js
+++ b/src/components/src/action-card/index.js
@@ -72,6 +72,7 @@ class ActionCard extends Component {
 			hasWhiteHeader,
 			isPending,
 			expandable = false,
+			isButtonEnabled = false
 		} = this.props;
 
 		const { expanded } = this.state;
@@ -173,7 +174,7 @@ class ActionCard extends Component {
 									</Handoff>
 								) : onClick || hasInternalLink ? (
 									<Button
-										disabled={ disabled }
+										disabled={ disabled && ! isButtonEnabled }
 										isLink
 										href={ href }
 										onClick={ onClick }
diff --git a/src/wizards/advertising/components/add-ons/index.js b/src/wizards/advertising/components/add-ons/index.js
index 456a1f68a9..19e70ff983 100644
--- a/src/wizards/advertising/components/add-ons/index.js
+++ b/src/wizards/advertising/components/add-ons/index.js
@@ -25,11 +25,13 @@ const MediaKitToggle = () => {
 		newspack_ads_wizard.media_kit_page_status
 	);
 
+	const isPagePublished = pageStatus === 'publish';
+
 	const toggleMediaKit = () => {
 		setInFlight( true );
 		apiFetch( {
 			path: '/newspack/v1/wizard/billboard/media-kit',
-			method: pageStatus === 'publish' ? 'DELETE' : 'POST',
+			method: isPagePublished ? 'DELETE' : 'POST',
 		} )
 			.then( ( { edit_url, page_status } ) => {
 				setEditURL( edit_url );
@@ -41,12 +43,14 @@ const MediaKitToggle = () => {
 	const props = editURL
 		? {
 				href: editURL,
-				actionText: __( 'Edit Media Kit page', 'newspack-plugin' ),
+				actionText: isPagePublished
+					? __( 'Edit Media Kit page', 'newspack-plugin' )
+					: __( 'Review draft page', 'newspack-plugin' ),
 		  }
 		: {};
 
 	let description = __(
-		'Media Kit page is not published. Click the link to edit and publish it.',
+		'Media kit page is created but unpublished, click the link to review and publish.',
 		'newspack-plugin'
 	);
 	let toggleEnabled = false;
@@ -70,10 +74,11 @@ const MediaKitToggle = () => {
 	return (
 		<ActionCard
 			disabled={ isInFlight || ! toggleEnabled }
+			isButtonEnabled={ true }
 			title={ __( 'Media Kit', 'newspack-plugin' ) }
 			description={ description }
 			toggle
-			toggleChecked={ Boolean( editURL ) }
+			toggleChecked={ Boolean( editURL ) && isPagePublished }
 			toggleOnChange={ toggleMediaKit }
 			{ ...props }
 		/>