From 5a021e63a19628acfe9eb660112214c1ed658faa Mon Sep 17 00:00:00 2001
From: Goffi <goffi@goffi.org>
Date: Wed, 29 Jun 2022 12:13:49 +0200
Subject: [PATCH] CLI (identity): add description to `identity set`:

rel 368
---
 sat_frontends/jp/cmd_identity.py | 17 +++++++++++++----
 1 file changed, 13 insertions(+), 4 deletions(-)

diff --git a/sat_frontends/jp/cmd_identity.py b/sat_frontends/jp/cmd_identity.py
index de33aa67..81fed6b6 100644
--- a/sat_frontends/jp/cmd_identity.py
+++ b/sat_frontends/jp/cmd_identity.py
@@ -72,15 +72,24 @@ class Set(base.CommandBase):
             "-n",
             "--nickname",
             action="append",
+            metavar="NICKNAME",
             dest="nicknames",
-            required=True,
             help=_("nicknames of the entity"),
         )
+        self.parser.add_argument(
+            "-d",
+            "--description",
+            help=_("description of the entity"),
+        )
 
     async def start(self):
-        id_data = {
-            "nicknames": self.args.nicknames,
-        }
+        id_data = {}
+        for field in ("nicknames", "description"):
+            value = getattr(self.args, field)
+            if value is not None:
+                id_data[field] = value
+        if not id_data:
+            self.parser.error("At least one metadata must be set")
         try:
             self.host.bridge.identitySet(
                 data_format.serialise(id_data),