You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/10/01 13:56:27 UTC

[GitHub] [airflow] Jorricks commented on pull request #26457: Add user comment to task instance and dag run

Jorricks commented on PR #26457:
URL: https://github.com/apache/airflow/pull/26457#issuecomment-1264371876

   > * 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.
   > 
   > We should still change the gradient to go to transparent instead of black.
   
   Absolutely love the refactoring! Even more new node.js syntax I didn't know about at all xD.
   Updated the ToDo list.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org