You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by pi...@apache.org on 2024/09/13 09:29:20 UTC

(airflow) branch main updated: fix: only treat null/undefined as falsy when rendering XComEntry (#42199)

This is an automated email from the ASF dual-hosted git repository.

pierrejeambrun pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/main by this push:
     new ade9de12de fix: only treat null/undefined as falsy when rendering XComEntry (#42199)
ade9de12de is described below

commit ade9de12defdd2727d3308c7e34b28ba4f68d18f
Author: Detlev V. <de...@protonmail.com>
AuthorDate: Fri Sep 13 11:29:13 2024 +0200

    fix: only treat null/undefined as falsy when rendering XComEntry (#42199)
    
    fix: only treat null, undefined or NaN as NULL in XComEntry render
    
    fix: use coalescing nullish check + isNaN
    
    fix: only treat null/undefined as falsy when rendering XComEntry
---
 airflow/www/static/js/dag/details/taskInstance/Xcom/XcomEntry.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/airflow/www/static/js/dag/details/taskInstance/Xcom/XcomEntry.tsx b/airflow/www/static/js/dag/details/taskInstance/Xcom/XcomEntry.tsx
index 806209bccb..4e2b9dc2dc 100644
--- a/airflow/www/static/js/dag/details/taskInstance/Xcom/XcomEntry.tsx
+++ b/airflow/www/static/js/dag/details/taskInstance/Xcom/XcomEntry.tsx
@@ -67,7 +67,7 @@ const XcomEntry = ({
         No value found for XCom key
       </Alert>
     );
-  } else if (!xcom.value) {
+  } else if (xcom.value === undefined || xcom.value === null) {
     content = (
       <Alert status="info">
         <AlertIcon />