bbovenzi commented on code in PR #26457:
URL: https://github.com/apache/airflow/pull/26457#discussion_r981340425
##########
airflow/www/static/js/dag/details/taskInstance/index.tsx:
##########
@@ -90,6 +92,8 @@ const TaskInstance = ({
if (!group || !run || !instance) return null;
+ const updateTaskInstanceNotesCallable = (newNotes: string) => {
instance.notes = newNotes; };
Review Comment:
Ahh yes, we should add an `onSuccess` callback to the set notes API
functions. In that function we get the grid data and find the instance in
question and update its note. Unfortunately, that is quite a roundabout way to
do it, but it ensures that the data is properly updated.
I can implement that part if you want.
--
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]
