Merge pull request #1 from contentstack/warning-message-changes

Warning Message Changes

Author: rohitmishra209

app.js

@@ -1,7 +1,7 @@
 const Bluebird = require('bluebird');
 const prompt = require('prompt');
 const chalk = require('chalk');
-const Confirm = require('prompt-confirm');
+//const Confirm = require('prompt-confirm');
 
 const util = require('./lib/util');
 const login = require('./lib/util/login');
@@ -25,16 +25,10 @@ login(config).then(function() {
 
         prompt.start();
 
-        var message = chalk.red("Please Enter your Stack name, From which you want to delete ?")
+        var message = chalk.red("To confirm the delete action, enter your stack name ?")
 
         prompt.get([message], function(err, result) {
             if (result[message] === config.stack.name) {
-            const confirm = new Confirm('The utility will delete all the Content Types and Assets present in the stack that you provide here.Are you sure, you want to continue')
-            confirm
-            .run()
-            .then(function(answer) {
-            // eslint-disable-next-line no-empty
-            if(answer === true) { 
                  // eslint-disable-next-line no-undef
                  if (process.argv.length === 3) {
                     // eslint-disable-next-line no-undef
@@ -45,7 +39,7 @@ login(config).then(function() {
                             log.success( 'Deletion completed successfully!');
                             return;
                         }).catch(function(error) {
-                            log.error('Failed to Delete ' + val + error);
+                            log.error('Failed to Delete ' +" "+JSON.stringify(error));
                            // log.error(error);
                             return;
                         })
@@ -73,16 +67,9 @@ login(config).then(function() {
                     log.error('Only one module can be deleted at a time.');
                     return 0;
                 }  
-            } else {
-                return 0;       
-            }
-            }).catch(function (error) {
-            // eslint-disable-next-line no-console
-            return(error);
-            }); 
             } else {
                 // eslint-disable-next-line no-console
-                console.log("You have Entered Wrong Stack Name");
+                console.log("Failed. Stack name does not match with stack detail specified in the config file.");
                 log.error(err)
                 return 0;
             }

lib/delete/delete_assets.js

@@ -30,7 +30,7 @@ DeleteAseets.prototype = {
     log.success(chalk.blue('Starting Assets deletion'));
     var self = this;
       return new Promise(function (resolve, reject) {
-        const confirm = new Confirm('Are you sure, you want to delete All Assets')
+        const confirm = new Confirm(chalk.red('The action will delete all the Assets present in the mentioned stack. Are you sure you want to continue ?'))
         confirm
         .run()
         .then(function(answer) {
@@ -43,7 +43,7 @@ DeleteAseets.prototype = {
             })
           } else {
             // eslint-disable-next-line no-console
-            console.log("deleteAssets is not set as true in config file or you have selected No")
+            console.log(chalk.red("Delete action failed. Either 'assetsdelete' is not set as 'true' in the config file or you declined while confirming the delete action."))
             return reject();
           }
         }).catch(function (error) {

lib/delete/delete_contentTypes.js

@@ -32,7 +32,7 @@ DeleteContentTypes.prototype = {
       var contenttypeslist = config.content_types_list;
       self.contentTypeList = contenttypeslist
       if(contenttypeslist && contenttypeslist != '' && contenttypeslist != undefined) {
-      const confirm = new Confirm('Are you sure, you want to delete'+" "+"'"+contenttypeslist+"'"+" "+'contentTypes')
+      const confirm = new Confirm(chalk.red('The action will delete the following Content Types.' +"'"+contenttypeslist+"'"+'.Are you sure you want to continue:?'))
         confirm
         .run()
         .then(function(answer) {
@@ -59,7 +59,7 @@ DeleteContentTypes.prototype = {
           return reject(error);
         }); 
       } else {
-        const confirm = new Confirm('Are you sure, you want to delete All ContentTypes of'+" "+config.stack.name+" "+"stack")
+        const confirm = new Confirm(chalk.red('The action will delete all the Content Types present in the mentioned stack. Are you sure you want to continue ?'))
         confirm
         .run()
         .then(function(answer) {
@@ -156,7 +156,7 @@ DeleteContentTypes.prototype = {
         concurrency: 1
       }).then(function () {
         return resolve();
-      }).catch(function (error) {
+      }).catch(function () {
         return reject();
       })
     }) 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "contentstack-bulk-delete",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "Contentstack's Bulk Delete Utility allows you to perform bulk delete operations. That means, that it is possible to delete all content types or a specific content type(s), and all assets in just one operation.",
   "homepage": "",
   "author": {