Skip to content

Fix the dags count filter in webserver home page - #34944

Merged
eladkal merged 2 commits into
apache:mainfrom
hussein-awala:fix_broken_filter
Oct 15, 2023
Merged

Fix the dags count filter in webserver home page#34944
eladkal merged 2 commits into
apache:mainfrom
hussein-awala:fix_broken_filter

Conversation

@hussein-awala

Copy link
Copy Markdown
Member

This PR revert a change added by mistake to #33810, this change introduced a bug in the dags count in the webserver home page. (b470c6b#r129957716)

@potiukpotiuk added this to the Airflow 2.7.3 milestone Oct 14, 2023
@potiuk

Copy link
Copy Markdown
Member

I wonder if we could have a unit test for that one to avoid similar brekages?

@hussein-awala

Copy link
Copy Markdown
MemberAuthor

I wonder if we could have a unit test for that one to avoid similar brekages?

I just added a test for it, it fails without this change.

@eladkal
eladkal merged commit 54e7e08 into apache:mainOct 15, 2023
@ephraimbuddyephraimbuddy added the type:bug-fix Changelog: Bug Fixes label Oct 27, 2023
ephraimbuddy pushed a commit that referenced this pull request Oct 29, 2023
* Fix the dags count filter in webserver home page
* Add a test to avoid breaking the count in the future
(cherry picked from commit 54e7e08)
ephraimbuddy pushed a commit that referenced this pull request Oct 30, 2023
* Fix the dags count filter in webserver home page
* Add a test to avoid breaking the count in the future
(cherry picked from commit 54e7e08)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:webserverWebserver related Issuestype:bug-fixChangelog: Bug Fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@hussein-awala@potiuk@ephraimbuddy@eladkal