jedcunningham opened a new pull request, #74746: URL: https://github.com/apache/airflow/pull/74746
Tidies the disclosure pills in the Dag version comparison header (Versions tab): - Removed the static "Observed state" pill; the ⓘ tooltip still explains that this compares stored state, not source. - Removed the "Values shown" pill. - The "Values hidden" pill now appears only when the server withholds values (the caller lacks Dag code access), and "Values require access to Dag code" moved from inline text to that pill's hover tooltip. Previously the hint showed even to users who had that access. Checked manually with the FAB auth manager: an Admin user sees values and no pill; a user with Viewer permissions minus "can read on DAG Code" gets the redacted diff and the "Values hidden" pill with its tooltip. Values visible, before: <img width="1201" height="395" alt="Screenshot 2026-10-14 at 9 41 44 AM" src="https://github.com/user-attachments/assets/50747c2d-1362-46f9-afea-20b756f95b3b" /> Values visible, after: <img width="1198" height="338" alt="Screenshot 2026-10-14 at 9 41 59 AM" src="https://github.com/user-attachments/assets/c3db8877-05ae-4d34-8ace-995f2e12075e" /> Values hidden (user without DAG Code read), after, hovering the pill. Before, this showed "Observed state" + "Values hidden" + the hint inline: <img width="1207" height="377" alt="Screenshot 2026-10-14 at 9 42 11 AM" src="https://github.com/user-attachments/assets/8aa4cd8b-72f7-4c01-8fa0-6841a6d0eca1" /> -- 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]
