bbovenzi commented on PR #26457: URL: https://github.com/apache/airflow/pull/26457#issuecomment-1263988769
> Now that @bbovenzi did a huge improvement to the update step of the API, what is left? > > * We still want to write a test for `setDagTaskNote.tsx`. > * Change the name according to the comment of @bbovenzi. > * Fix the failing test. > > Besides that, do we need to do anything else? For context; Going on holiday soon and would like to get this ready beforehand :). - I added react tests for the note component. I also renamed it, generally functions start with verbs while components do not. - There were also some console warnings from a rebase mistake. - Finally, I moved around the set notes mutations to accept notes in the mutateAsync function instead of the API hook itself. -- 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]
