Re: [PR] Airflow variable include description field in variable import/export UI [airflow]

["dirrao (via GitHub)" <gi...@apache.org>]

dirrao commented on code in PR #39075:
URL: https://github.com/apache/airflow/pull/39075#discussion_r1570326833


##########
airflow/www/views.py:
##########
@@ -4547,9 +4547,11 @@ def action_varexport(self, items):
         for var in items:
             try:
                 val = decoder.decode(var.val)
+                description = decoder.decode(var.description)
             except Exception:
                 val = var.val
-            var_dict[var.key] = val
+                description = var.description
+            var_dict[var.key] = {"val": val, "description":description}

Review Comment:
   Description is optional thing. Looks like this code doesn't cover description optional thing.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org