Skip to content

Add description field for filter_param_factory#54903

Merged
jason810496 merged 3 commits intoapache:mainfrom
jason810496:feature/AIP-84/add-description-for-filter_param_factory
Sep 2, 2025
Merged

Add description field for filter_param_factory#54903
jason810496 merged 3 commits intoapache:mainfrom
jason810496:feature/AIP-84/add-description-for-filter_param_factory

Conversation

@jason810496
Copy link
Member

related: add has_import_errors filter to Core API GET /dags endpoint #54563

What

By adding optional description field to filter_param_factory, it will be easy to add description for any query parameter.

Like the case in #54563 (comment)

So I think it's still useful to create such filter. I also suggest to update any related doc,
to highlight the Import Errors filter only returns DAGs that have been successfully loaded before.

@boring-cyborg boring-cyborg bot added the area:API Airflow's REST/HTTP API label Aug 25, 2025
@jason810496 jason810496 force-pushed the feature/AIP-84/add-description-for-filter_param_factory branch from 16f1a4d to 9331f7c Compare August 25, 2025 12:33
Copy link
Member

@Lee-W Lee-W left a comment

Choose a reason for hiding this comment

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

should we add a test here?

@jason810496 jason810496 force-pushed the feature/AIP-84/add-description-for-filter_param_factory branch from 9331f7c to c29696c Compare September 2, 2025 12:40
@jason810496 jason810496 merged commit ab1e0c2 into apache:main Sep 2, 2025
107 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:API Airflow's REST/HTTP API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants