proca 2.1.0 → 2.1.1

package/README.md

@@ -1980,8 +1980,6 @@ DESCRIPTION
   Update a widget's properties
 
 EXAMPLES
-  $ proca widget update -i 42 --rename campaign/new_name
-
   $ proca widget update -name campaign/widget --locale fr
 
   $ proca widget update 42 --confirm-optin

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "proca",
   "description": "Access the proca api",
-  "version": "2.1.0",
+  "version": "2.1.1",
   "author": "Xavier",
   "bin": {
     "proca": "proca-cli"

package/src/commands/campaign/widget/archive.mjs

@@ -2,7 +2,7 @@ import { Flags } from "@oclif/core";
 import prompts from "prompts";
 import CampaignGet from "#src/commands/campaign/get.mjs";
 import WidgetList from "#src/commands/widget/list.mjs";
-import WidgetUpdate from "#src/commands/widget/update.mjs";
+import WidgetUpdate from "#src/commands/widget/update/index.mjs";
 import Command from "#src/procaCommand.mjs";
 
 export default class CampaignWidgetArchive extends Command {

package/src/commands/widget/get.mjs

@@ -10,7 +10,6 @@ export const getWidget = (params) => {
 
 export default class WidgetGet extends Command {
   static description = "view a widget";
-
   static args = this.multiid();
 
   static flags = {

package/src/commands/widget/update/external.mjs

@@ -2,6 +2,11 @@ import { Flags } from "@oclif/core";
 import Command from "#src/procaCommand.mjs";
 import { gql, mutation } from "#src/urql.mjs";
 
+export const updateCounter = (id, counter) => {
+  const d = new CounterUpdate([]);
+  return d.updateCounter(id, counter);
+};
+
 export default class CounterUpdate extends Command {
   static description =
     "Update the global counter to add the actions collected elsewhere";

package/src/commands/widget/{update.mjs → update/index.mjs}

@@ -6,9 +6,7 @@ import { gql, mutation } from "#src/urql.mjs";
 
 export default class WidgetUpdate extends Command {
   static description = "Update a widget's properties";
-
   static examples = [
-    "<%= config.bin %> <%= command.id %> -i 42 --rename campaign/new_name",
     "<%= config.bin %> <%= command.id %> -name campaign/widget --locale fr",
     "<%= config.bin %> <%= command.id %> 42 --confirm-optin",
     "<%= config.bin %> <%= command.id %> --dxid=pnc -confirm-optin --dry-run",
@@ -16,8 +14,8 @@ export default class WidgetUpdate extends Command {
 
   static args = this.multiid();
 
+  // @ivana, let's make separate update xxx than inherit, check external
   static flags = {
-    // flag with no value (-f, --force)
     ...this.flagify({ multiid: true }),
     rename: Flags.string({
       hidden: true, // use proca widget update name instead

package/src/commands/widget/update/name.mjs

@@ -1,6 +1,6 @@
 import { Flags } from "@oclif/core";
 import { getWidget } from "#src/commands/widget/get.mjs";
-import UpdateCommand from "#src/commands/widget/update.mjs";
+import UpdateCommand from "#src/commands/widget/update/index.mjs";
 
 export default class CounterUpdate extends UpdateCommand {
   static description = "Update the name of a widget";

package/src/procaCommand.mjs

@@ -100,10 +100,11 @@ class ProcaCommand extends Command {
     ].filter(Boolean).length;
 
     if (identified === 0) {
-      this.error("One of --name, --id, or --dxid is required");
+      super.error("One of --name, --id, or --dxid is required", {
+        code: 1,
+      });
     }
 
-    await super.parse(); // check that either the first arg or the name/id/dxid are set
     return parsed;
   }
 
@@ -143,14 +144,14 @@ class ProcaCommand extends Command {
 
     if (err.networkError) {
       this.info("Looks like there’s a problem with your internet connection");
-      this.error(err.networkError.cause, { exit: err.exitCode || 1 });
+      this.error(err.networkError.cause, { exit: err.code || 1 });
     }
     if (err instanceof SyntaxError) {
-      this.error(`Syntax error: ${err.message}`, { exit: 1 });
+      this.error(`Syntax error: ${err.message}`, { code: 1 });
     }
 
     // Default error handling
-    this.error(err.message, { exit: err.exitCode || 1 });
+    this.error(err.message, { exit: err.code || 1 });
   }
 
   flatten = (obj, prefix = "", result = {}) => {