From d342cb57ea5c6bbe378e49f3aca5faff4d6ab8c1 Mon Sep 17 00:00:00 2001
From: Aman Kumar <aman.kumar@contentstack.com>
Date: Wed, 26 Jun 2024 13:06:58 +0530
Subject: [PATCH] refactor: deploy command ui text

---
 README.md                  | 15 +++++++--------
 src/commands/app/deploy.ts |  6 ++++--
 src/messages/index.ts      | 18 +++++++++---------
 src/util/inquirer.ts       |  2 +-
 4 files changed, 21 insertions(+), 20 deletions(-)

diff --git a/README.md b/README.md
index c2d0b1f..a4501a3 100644
--- a/README.md
+++ b/README.md
@@ -21,7 +21,6 @@ $ csdx COMMAND
 running command...
 $ csdx (--version|-v)
 @contentstack/apps-cli/1.2.0 darwin-arm64 node-v18.20.2
-@contentstack/apps-cli/1.2.0 darwin-arm64 node-v18.20.2
 $ csdx --help [COMMAND]
 USAGE
   $ csdx COMMAND
@@ -130,16 +129,16 @@ Deploy an app
 
 ```
 USAGE
-  $ csdx app:deploy [--app-uid <value>] [--hosting-type <value>] [--app-url <value>] [-y] [--launch-project
-    existing|new] [-c <value>]
+  $ csdx app:deploy [--app-uid <value>] [--hosting-type Hosting with Launch|Custom Hosting] [--app-url <value>]
+    [--launch-project existing|new] [-c <value>]
 
 FLAGS
-  -c, --config=<value>       [optional] path of config file
-  -y, --yes                  Force disconnect launch project by skipping the confirmation
+  -c, --config=<value>       [optional] Please enter the path of the config file.
   --app-uid=<value>          Provide the app UID of an existing app.
-  --app-url=<value>          App URL
-  --hosting-type=<value>     Hosting Type
-  --launch-project=<option>  Launch Project
+  --app-url=<value>          Please enter the URL of the app you want to deploy.
+  --hosting-type=<option>    Please provide a valid Hosting Type.
+                             <options: Hosting with Launch|Custom Hosting>
+  --launch-project=<option>  Choose a new or an existing Launch project.
                              <options: existing|new>
 
 DESCRIPTION
diff --git a/src/commands/app/deploy.ts b/src/commands/app/deploy.ts
index 0916952..585ce80 100644
--- a/src/commands/app/deploy.ts
+++ b/src/commands/app/deploy.ts
@@ -41,6 +41,8 @@ export default class Deploy extends AppCLIBaseCommand {
       description: commonMsg.APP_UID,
     }),
     "hosting-type": Flags.string({
+      multiple: false,
+      options: ["Hosting with Launch", "Custom Hosting"],
       description: deployAppMsg.HOSTING_TYPE,
     }),
     "app-url": Flags.string({
@@ -90,7 +92,7 @@ export default class Deploy extends AppCLIBaseCommand {
           await this.handleHostingWithLaunch(config, updateHostingPayload, projects);
           break;
         default:
-          this.log("Invalid hosting type", "error");
+          this.log("Please provide a valid Hosting Type.", "error");
           return;
       }
 
@@ -197,7 +199,7 @@ export default class Deploy extends AppCLIBaseCommand {
       );
       url = await this.handleNewProject(config, updateHostingPayload);
     } else {
-      this.log("Invalid project type", "error");
+      this.log("Please provide a valid launch project.", "error");
       return;
     }
 
diff --git a/src/messages/index.ts b/src/messages/index.ts
index e77d3a3..7007cce 100644
--- a/src/messages/index.ts
+++ b/src/messages/index.ts
@@ -113,16 +113,16 @@ const reinstallAppMsg = {
 }
 
 const deployAppMsg = {
-  APP_DEPLOYED: "{app} deployed successfully.",
+  APP_DEPLOYED: "{app} has been deployed successfully.",
   FORCE_DISCONNECT: "Force disconnect launch project by skipping the confirmation",
-  LAUNCH_PROJECT: "Launch Project",
-  APP_URL: "App URL",
-  HOSTING_TYPE: "Hosting Type",
-  CONFIG_FILE: "[optional] path of config file",
-  APP_UPDATE_TERMINATION_MSG: "Launch Project connected! Skipping app hosting updates process...",
-  DISCONNECT_PROJECT: "Are you sure you want to disconnect launch Project ?",
-  PROJECT_NOT_FOUND: "Project not found!",
-  PROJECT_NAME_CONFLICT_FAILED: "Project name conflict resolution failed!"
+  LAUNCH_PROJECT: "Choose a new or an existing Launch project.",
+  APP_URL: "Please enter the URL of the app you want to deploy.",
+  HOSTING_TYPE: "Please provide a valid Hosting Type.",
+  CONFIG_FILE: "[optional] Please enter the path of the config file.",
+  APP_UPDATE_TERMINATION_MSG: "The app is already connected to the Launch project. Skipping the app hosting updates process.",
+  DISCONNECT_PROJECT: "Are you sure you wan to disconnect the Launch project?",
+  PROJECT_NOT_FOUND: "We couldn't find the project. Please enter a valid project name.",
+  PROJECT_NAME_CONFLICT_FAILED: "Unable to resolve the project name conflict."
 }
 
 const messages: typeof errors &
diff --git a/src/util/inquirer.ts b/src/util/inquirer.ts
index f28db07..455b33c 100644
--- a/src/util/inquirer.ts
+++ b/src/util/inquirer.ts
@@ -373,7 +373,7 @@ const askProjectName = async (
     type: "input",
     name: "name",
     validate: inquireRequireValidation,
-    message: `${projectName} project already exist. Enter a new name to create a project.?`,
+    message: `The project name ${projectName} is already in use. Please enter a different name for your project`,
   });
 };