Jasperora commented on PR #53307:
URL: https://github.com/apache/airflow/pull/53307#issuecomment-3124482882

   Sorry for the late reply. I would like to confirm my understanding regarding 
the modal and state management.
   
   Currently, the `mdContent` prop for `EditableMarkdownButton` is derived from 
the `note` state in either `Run/Header.tsx` or `TaskInstance/Header.tsx`. 
Unmounting the modal alone does not reset or update the `note` state in the 
parent component, so even if the modal is unmounted, `mdContent` may still 
reflect an outdated value if `note` was changed elsewhere.
   
   To ensure that the modal always displays the most current content, it seems 
necessary to use an `onOpen` function to set the `note` state each time the 
modal is opened. This approach would guarantee that the initial state is 
refreshed appropriately.
   
   Please let me know if my understanding is correct or not. Thank you again 
for your guidance.


-- 
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]

Reply via email to