Skip to content

Add Has Events filter to Assets search - #72460

Open
jasperjonkhans wants to merge 1 commit into
apache:mainfrom
jasperjonkhans:issue-72281
Open

Add Has Events filter to Assets search#72460
jasperjonkhans wants to merge 1 commit into
apache:mainfrom
jasperjonkhans:issue-72281

Conversation

@jasperjonkhans

@jasperjonkhans jasperjonkhans commented Sep 2, 2026 •
edited
Loading

Copy link
Copy Markdown

Assets search now supports a tri-state has_events query parameter and exposes it as a URL-backed Has Events filter in the UI. This lets users distinguish assets with recorded events from definitions that have never emitted one and preserve or share the filtered view.

The change:

  • filters assets through the distinct set of asset IDs referenced by asset events
  • adds Yes, No, and unset UI states
  • regenerates the private UI OpenAPI specification and TypeScript clients
  • covers all three states in API and UI tests

closes: #72281

Was generative AI tooling used to co-author this PR?
  • Yes — assisted by Pi Coding Agent 0.84.4 using gpt-5.6-sol max effort

Generated-by: Pi Coding Agent 0.84.4 following the guidelines

@jasperjonkhans
Add Has Events filter to Assets search
aa6c307 
Large asset catalogs expose event activity only row by row, making it
difficult to distinguish live assets from definitions that have never
been updated.
@boring-cyborg boring-cyborg Bot added area:API Airflow's REST/HTTP API area:translations area:UI Related to UI/UX. For Frontend Developers. translation:default labels Sep 2, 2026
@fat-catTW

fat-catTW commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Thanks, LGTM!!

Just one nit.
Since this is a user-facing airflow-core feature, could you add a newsfragment under airflow-core/newsfragments/, probably as a .feature.rst or .improvement.rst?

No behavior blocker from my side beyond that.

@ColtenOuO ColtenOuO mentioned this pull request Sep 5, 2026
Open
ColtenOuO
ColtenOuO approved these changes Sep 5, 2026
View reviewed changes

@ColtenOuO ColtenOuO left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, Great work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bbovenzi bbovenzi Awaiting requested review from bbovenzi bbovenzi is a code owner
@pierrejeambrun pierrejeambrun Awaiting requested review from pierrejeambrun pierrejeambrun is a code owner
@ryanahamilton ryanahamilton Awaiting requested review from ryanahamilton ryanahamilton is a code owner
@shubhamraj-git shubhamraj-git Awaiting requested review from shubhamraj-git shubhamraj-git is a code owner
@guan404ming guan404ming Awaiting requested review from guan404ming guan404ming is a code owner
@vatsrahul1001 vatsrahul1001 Awaiting requested review from vatsrahul1001 vatsrahul1001 is a code owner
@choo121600 choo121600 Awaiting requested review from choo121600 choo121600 is a code owner
@ephraimbuddy ephraimbuddy Awaiting requested review from ephraimbuddy ephraimbuddy is a code owner
@rawwar rawwar Awaiting requested review from rawwar rawwar is a code owner
@jason810496 jason810496 Awaiting requested review from jason810496 jason810496 is a code owner
@bugraoz93 bugraoz93 Awaiting requested review from bugraoz93 bugraoz93 is a code owner
@henry3260 henry3260 Awaiting requested review from henry3260 henry3260 is a code owner
1 more reviewer
@ColtenOuO ColtenOuO ColtenOuO approved these changes
Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

area:API Airflow's REST/HTTP API area:translations area:UI Related to UI/UX. For Frontend Developers. translation:default

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add a has_events filter to the Assets search view

3 participants

@jasperjonkhans @fat-catTW @ColtenOuO