fix: warnings

Author: ChiragAgg5k

templates/cli/lib/client.js.twig

@@ -136,11 +136,6 @@ class Client {
             },
         }),
       });
-
-      const warnings = response.headers.get('x-{{ spec.title | lower }}-warning');
-      if (warnings) {
-          warnings.split(';').forEach((warning) => console.log(`${chalk.yellow.bold("ℹ Warning:")} ${chalk.yellow(warning)}`));
-      }
     } catch (error) {
       throw new {{spec.title | caseUcfirst}}Exception(error.message);
     }

templates/cli/lib/commands/pull.js.twig

@@ -21,7 +21,7 @@ const pullResources = async () => {
         settings: pullSettings,
         functions: pullFunctions,
         sites: pullSites,
-        collections: pullCollection,
+        tables: pullTable,
         buckets: pullBucket,
         teams: pullTeam,
         messages: pullMessagingTopic
@@ -286,6 +286,7 @@ const pullSites = async ({ code, withVariables }) => {
 }
 
 const pullCollection = async () => {
+    warn("appwrite push collection has been deprecated. Please consider using 'appwrite push tables' instead");
     log("Fetching collections ...");
     let total = 0;
 
@@ -500,7 +501,7 @@ pull
 pull
     .command("collection")
     .alias("collections")
-    .description("Pull your {{ spec.title|caseUcfirst }} collections")
+    .description("Pull your {{ spec.title|caseUcfirst }} collections (deprecated, please use 'pull tables' instead)")
     .action(actionRunner(pullCollection))
 
 pull

templates/cli/lib/commands/push.js.twig

@@ -1819,7 +1819,7 @@ const pushTable = async ({ returnOnZero, attempts } = { returnOnZero: false }) =
 }
 
 const pushCollection = async ({ returnOnZero, attempts } = { returnOnZero: false }) => {
-    warn("⚠️  DEPRECATED: Use 'appwrite push table' instead. This command might be removed in the future");
+    warn("appwrite push collection has been deprecated. Please consider using 'appwrite push tables' instead");
     const collections = [];
 
     if (attempts) {
@@ -1882,7 +1882,7 @@ const pushCollection = async ({ returnOnZero, attempts } = { returnOnZero: false
     }));
 
 
-    if (!(await approveChanges(collections, databasesGetCollection, KeysCollection, 'collectionId', 'collections', ['columns', 'indexes'], 'databaseId', 'databaseId',))) {
+    if (!(await approveChanges(collections, databasesGetCollection, KeysCollection, 'collectionId', 'collections', ['attributes', 'indexes'], 'databaseId', 'databaseId',))) {
         return;
     }
     // Parallel collection actions
@@ -2227,20 +2227,20 @@ push
     .option("--with-variables", `Push site variables.`)
     .action(actionRunner(pushSite));
 
+push
+    .command("collection")
+    .alias("collections")
+    .description("Push collections in the current project. (deprecated, please use 'push tables' instead)")
+    .option(`-a, --attempts <numberOfAttempts>`, `Max number of attempts before timing out. default: 30.`)
+    .action(actionRunner(pushCollection));
+
 push
     .command("table")
     .alias("tables")
     .description("Push tables in the current project.")
     .option(`-a, --attempts <numberOfAttempts>`, `Max number of attempts before timing out. default: 30.`)
     .action(actionRunner(pushTable));
 
-push
-    .command("collection")
-    .alias("collections")
-    .description("Push collections in the current project.")
-    .option(`-a, --attempts <numberOfAttempts>`, `Max number of attempts before timing out. default: 30.`)
-    .action(actionRunner(pushCollection));
-
 push
     .command("bucket")
     .alias("buckets")