jscheffl commented on PR #44869: URL: https://github.com/apache/airflow/pull/44869#issuecomment-2538718473
> Thanks @jscheffl, it needs porting `ReactJSON` related component from legacy UI and would like to tackle it in a separate PR as noted in the PR description since it seems to take fair bit of work. Fair. As long as it is not forgotten :-D And does not need to be e exactly the same component as in legacy if there are better components nowadays. Regarding (3) in your list above: In the past "Rest API" call the public XCom endpoint was used and for adding the "ReactJSON" I needed to add the "stringify" option as workarouns as the stringified version before used Python-style quotes which were not JSON parsable in the React / Javascript code. In the legacy this is really bad (in my view) and it would be better if in the new UI it is made "right" from the beginning. I wanted to prevent this in the past not having a breaking change in the REST API - but now would be the time with FastAPI anyway. So `stringify` might need to be set to `false` later back again to get a native object, else would be a waste to re-parse the string to JSON in TypeScript again after REST endpoint -- 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]
