From 2a6e98c27ff8a2506873fab460d4fbede1e6cd3f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Cl=C3=A9ment=20Pasteau?=
 <4895034+ClementPasteau@users.noreply.github.com>
Date: Fri, 22 Nov 2024 09:57:46 +0100
Subject: [PATCH] Update publish flow to allow creating links if game is not
 owned (#7184)

---
 .../OnlineWebExport/OnlineGameLink.js         |  6 ++--
 .../OnlineWebExport/OnlineWebExportFlow.js    | 35 ++++++++++++-------
 .../OnlineWebExport/ShareOnlineGameDialog.js  |  9 ++---
 3 files changed, 30 insertions(+), 20 deletions(-)

diff --git a/newIDE/app/src/ExportAndShare/GenericExporters/OnlineWebExport/OnlineGameLink.js b/newIDE/app/src/ExportAndShare/GenericExporters/OnlineWebExport/OnlineGameLink.js
index 5443b0ca9d45..4bc6e66120d5 100644
--- a/newIDE/app/src/ExportAndShare/GenericExporters/OnlineWebExport/OnlineGameLink.js
+++ b/newIDE/app/src/ExportAndShare/GenericExporters/OnlineWebExport/OnlineGameLink.js
@@ -194,10 +194,10 @@ const OnlineGameLink = ({
               </Line>
             </Column>
           )}
-          {isShareDialogOpen && game && (
+          {isShareDialogOpen && (
             <ShareOnlineGameDialog
               gameThumbnailUrl={gameThumbnailUrl}
-              gameName={game.gameName}
+              gameName={game ? game.gameName : undefined} // This can happen if the game is not owned.
               buildOrGameUrl={buildOrGameUrl}
               isBuildPublished={!!isBuildPublished}
               loadingText={
@@ -209,7 +209,7 @@ const OnlineGameLink = ({
                 )
               }
               onClose={() => setIsShareDialogOpen(false)}
-              onOpenGameDashboard={openGameDashboard}
+              onOpenGameDashboard={game ? openGameDashboard : undefined}
             />
           )}
         </>
diff --git a/newIDE/app/src/ExportAndShare/GenericExporters/OnlineWebExport/OnlineWebExportFlow.js b/newIDE/app/src/ExportAndShare/GenericExporters/OnlineWebExport/OnlineWebExportFlow.js
index 6dbe5ecf4d6b..24c7e54b470c 100644
--- a/newIDE/app/src/ExportAndShare/GenericExporters/OnlineWebExport/OnlineWebExportFlow.js
+++ b/newIDE/app/src/ExportAndShare/GenericExporters/OnlineWebExport/OnlineWebExportFlow.js
@@ -27,24 +27,31 @@ const OnlineWebExportFlow = ({
   isExporting,
   uiMode,
 }: OnlineWebExportFlowProps) => {
-  const { game, builds, refreshGame, setGame } = gameAndBuildsManager;
-  const hasGameExistingBuilds =
+  const {
+    game,
+    builds,
+    refreshGame,
+    setGame,
+    gameAvailabilityError,
+  } = gameAndBuildsManager;
+  const isGameOwnedByCurrentUser =
+    !gameAvailabilityError || gameAvailabilityError !== 'not-owned';
+  const hasGameExistingWebBuilds =
     game && builds
-      ? !!builds.filter(build => build.gameId === game.id).length
+      ? !!builds.filter(
+          build => build.gameId === game.id && build.type === 'web-build'
+        ).length
       : false;
   const isPublishedOnGdGames = !!game && !!game.publicWebBuildId;
-  const webBuilds = builds
-    ? builds.filter(build => build.type === 'web-build')
-    : null;
-  const hasAlreadyPublishedOnWeb = webBuilds ? webBuilds.length > 0 : false;
 
-  const publishOnGdGamesByDefault =
-    // This check ensures the user has done the process of unpublishing the game from gd.games
-    !(hasAlreadyPublishedOnWeb && !isPublishedOnGdGames);
   const [
     automaticallyPublishNewBuild,
     setAutomaticallyPublishNewBuild,
-  ] = React.useState(publishOnGdGamesByDefault || !hasGameExistingBuilds);
+  ] = React.useState(
+    isGameOwnedByCurrentUser &&
+      (!hasGameExistingWebBuilds || // Never built for web? Let's publish automatically.
+        isPublishedOnGdGames) // Already published on GDevelop games? Let's update automatically.
+  );
 
   // Only show the buttons when the export is not started or when it's done.
   const shouldShowButtons =
@@ -60,7 +67,9 @@ const OnlineWebExportFlow = ({
       >
         <RaisedButton
           label={
-            !hasGameExistingBuilds ? (
+            !isGameOwnedByCurrentUser ? (
+              <Trans>Generate a link</Trans>
+            ) : !hasGameExistingWebBuilds ? (
               <Trans>Publish game</Trans>
             ) : automaticallyPublishNewBuild ? (
               <Trans>Publish new version</Trans>
@@ -74,7 +83,7 @@ const OnlineWebExportFlow = ({
           disabled={disabled}
         />
       </Line>
-      {hasGameExistingBuilds && (
+      {hasGameExistingWebBuilds && (
         <Line justifyContent="center">
           <Toggle
             toggled={automaticallyPublishNewBuild}
diff --git a/newIDE/app/src/ExportAndShare/GenericExporters/OnlineWebExport/ShareOnlineGameDialog.js b/newIDE/app/src/ExportAndShare/GenericExporters/OnlineWebExport/ShareOnlineGameDialog.js
index f3a92ef59273..3152ef249325 100644
--- a/newIDE/app/src/ExportAndShare/GenericExporters/OnlineWebExport/ShareOnlineGameDialog.js
+++ b/newIDE/app/src/ExportAndShare/GenericExporters/OnlineWebExport/ShareOnlineGameDialog.js
@@ -31,9 +31,9 @@ const styles = {
 type Props = {|
   buildOrGameUrl: ?string,
   gameThumbnailUrl: ?string,
-  gameName: string,
+  gameName: ?string,
   isBuildPublished: boolean,
-  onOpenGameDashboard: () => void,
+  onOpenGameDashboard?: () => void,
   onClose: () => void,
   loadingText: ?React.Node,
 |};
@@ -63,10 +63,11 @@ const ShareOnlineGameDialog = ({
           label={<Trans>Done</Trans>}
           primary
           onClick={onClose}
+          disabled={!!loadingText}
         />,
       ]}
       secondaryActions={[
-        buildOrGameUrl && !loadingText ? (
+        buildOrGameUrl && !loadingText && onOpenGameDashboard ? (
           <FlatButton
             key="publish"
             primary
@@ -83,7 +84,7 @@ const ShareOnlineGameDialog = ({
         <ColumnStackLayout noMargin expand>
           <Column noMargin>
             <ResponsiveLineStackLayout noMargin noResponsiveLandscape>
-              {gameThumbnailUrl && (
+              {gameThumbnailUrl && gameName && (
                 <GameThumbnail
                   background="light"
                   fullWidthOnMobile