jscheffl commented on PR #43367: URL: https://github.com/apache/airflow/pull/43367#issuecomment-2466409780
> @jscheffl So, can we do something like, we take as string, and while sending request, we can parse to record? Reason being I need to separately handle the record for codemirror, since it's expecting type as string. Yes, understand. Codemirror wants to have plain text. But on API an object is needed (today and preferrably keep it like this). So in between on trigger you need to fetch the Codemirror and JSON parse before POST to server... Anyway I'd think this is good to ensure valid data is sent. In the existing / legacy UI also whenever you left the CodeMirror editor the test was re-formatted. Don't know if you ever tried... if syntax errors there was an error marker being set and if JSON was valid, it was re-formatted by Codemirror. -- 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]
