Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add rendered templates tab. #46750

Merged
merged 2 commits into from
Feb 15, 2025
Merged

Conversation

tirkarthi
Copy link
Contributor

@tirkarthi tirkarthi commented Feb 14, 2025

Rendered templates are useful for debugging. This PR doesn't address syntax highlighting as per the template field extension which needs API change and other changes requested in #39527 that can be part of future enhancements once API change is available. This adds a basic table with copy button and stringified value as initial work. This doesn't need auto-refresh as it doesn't change after initial task instance rendering.

In the screenshot empty tuple is rendered because "()" string value is returned in the API.

image

@boring-cyborg boring-cyborg bot added the area:UI Related to UI/UX. For Frontend Developers. label Feb 14, 2025
@tirkarthi tirkarthi added the AIP-38 Modern Web Application label Feb 14, 2025
@tirkarthi tirkarthi force-pushed the rendered-template-fields branch from 50e9ccc to 2916b4f Compare February 15, 2025 03:34
@jscheffl jscheffl merged commit 7afc322 into apache:main Feb 15, 2025
35 checks passed
dantonbertuol pushed a commit to dantonbertuol/airflow that referenced this pull request Feb 17, 2025
* Add rendered templates tab.

* Wrap value inside code block.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AIP-38 Modern Web Application area:UI Related to UI/UX. For Frontend Developers.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants