Skip to content

[v3-4-test] Let plugin views be scoped to specific Assets (#74609) - #74992

Merged
vatsrahul1001 merged 1 commit into
v3-4-testfrom
backport-230cd8a-v3-4-test
Oct 18, 2026
Merged

vatsrahul1001 merged 1 commit into
v3-4-testfrom
backport-230cd8a-v3-4-test

Conversation

@vatsrahul1001

Copy link
Copy Markdown
Contributor

Manual backport of #74609 to v3-4-test. The automatic backport failed on a cherry-pick conflict, so this was cherry-picked by hand.

Conflict resolution: the only conflict was airflow-core/newsfragments/74278.feature.rst (modify/delete). That newsfragment has already been consumed into RELEASE_NOTES.rst on the release branch, so it was dropped. All code changes applied unchanged.


Was generative AI tooling used to co-author this PR?
  • Yes — Claude Code (Opus 4.8)

Generated-by: Claude Code (Opus 4.8) following the guidelines

Asset pages could not be scoped at all. `asset` resolved no record, so every
`applies_to` path on an asset view was unevaluable and skipped, and skipping
does not narrow anything -- an asset view configured with scoping showed on
every asset, which is the opposite of what its author asked for.

The asset is already fetched for those pages, so it costs nothing to make it a
root an `applies_to` path can name, alongside the Dag, Dag Run, Task and Task
Instance records. `AssetResponse` joins the models paths are validated against,
so an asset path gets the same misspelling check as any other.

(cherry picked from commit 230cd8a)
@boring-cyborg boring-cyborg Bot added area:plugins area:UI Related to UI/UX. For Frontend Developers. kind:documentation labels Oct 18, 2026
@vatsrahul1001 vatsrahul1001 added this to the Airflow 3.4.0 milestone Oct 18, 2026
@vatsrahul1001
vatsrahul1001 merged commit 8bc8112 into v3-4-test Oct 18, 2026
98 checks passed
@vatsrahul1001
vatsrahul1001 deleted the backport-230cd8a-v3-4-test branch October 18, 2026 12:33
vatsrahul1001 added a commit that referenced this pull request Oct 20, 2026
Asset pages could not be scoped at all. `asset` resolved no record, so every
`applies_to` path on an asset view was unevaluable and skipped, and skipping
does not narrow anything -- an asset view configured with scoping showed on
every asset, which is the opposite of what its author asked for.

The asset is already fetched for those pages, so it costs nothing to make it a
root an `applies_to` path can name, alongside the Dag, Dag Run, Task and Task
Instance records. `AssetResponse` joins the models paths are validated against,
so an asset path gets the same misspelling check as any other.

(cherry picked from commit 230cd8a)

Co-authored-by: Brent Bovenzi <brent@astronomer.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:plugins area:UI Related to UI/UX. For Frontend Developers. kind:documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants