jscheffl commented on code in PR #40640:
URL: https://github.com/apache/airflow/pull/40640#discussion_r1676414041


##########
airflow/www/static/js/dag/details/taskInstance/Xcom/XcomEntry.tsx:
##########
@@ -66,6 +67,10 @@ const XcomEntry = ({
         No value found for XCom key
       </Alert>
     );
+  } else {
+    const xcomStr = xcom?.value as string;
+    const xcomStrR = xcomStr.replace(/'/g, '"');

Review Comment:
   Even after a sleep-over have no better idea than the xcom... left a comment 
in the code as proposed.
   Made conditionals by checking xcom and value in the condition above, then no 
undefined/conditional regex needed.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to