Skip to content

Conversation

@choo121600
Copy link
Member

part of #58436

Adds routing coverage for incorrect URLs by directing unmatched paths to the ErrorPage component.
By adding a catch-all path: "*" route, users will now see a proper error page instead of an undefined UI state when navigating to invalid URLs.

Before

Screen.Recording.2025-11-21.at.2.22.11.PM.mov

After

Screen.Recording.2025-11-21.at.3.09.58.PM.mov

^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@boring-cyborg boring-cyborg bot added area:translations area:UI Related to UI/UX. For Frontend Developers. translation:default labels Nov 21, 2025
@choo121600 choo121600 changed the title Fix/404 Add error handling route to catch unmatched paths Nov 21, 2025
@choo121600 choo121600 changed the title Add error handling route to catch unmatched paths Improve 404 handling by adding fallback route for invalid URLs Nov 22, 2025
Copy link
Member

@pierrejeambrun pierrejeambrun left a comment

Choose a reason for hiding this comment

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

Nice one, thanks.

@pierrejeambrun pierrejeambrun added this to the Airflow 3.1.4 milestone Nov 24, 2025
@pierrejeambrun pierrejeambrun added the backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch label Nov 24, 2025
@pierrejeambrun pierrejeambrun merged commit 946871c into apache:main Nov 24, 2025
64 checks passed
github-actions bot pushed a commit to aws-mwaa/upstream-to-airflow that referenced this pull request Nov 24, 2025
… URLs (apache#58546)

* Add error handling route to catch unmatched paths

* Add custom 404 error message for invalid URLs
(cherry picked from commit 946871c)

Co-authored-by: Yeonguk Choo <choo121600@gmail.com>
github-actions bot pushed a commit that referenced this pull request Nov 24, 2025
… URLs (#58546)

* Add error handling route to catch unmatched paths

* Add custom 404 error message for invalid URLs
(cherry picked from commit 946871c)

Co-authored-by: Yeonguk Choo <choo121600@gmail.com>
@github-actions
Copy link

Backport successfully created: v3-1-test

Status Branch Result
v3-1-test PR Link

potiuk pushed a commit that referenced this pull request Nov 24, 2025
… URLs (#58546)

* Add error handling route to catch unmatched paths

* Add custom 404 error message for invalid URLs
(cherry picked from commit 946871c)

Co-authored-by: Yeonguk Choo <choo121600@gmail.com>
pierrejeambrun pushed a commit that referenced this pull request Nov 25, 2025
… URLs (#58546) (#58629)

* Add error handling route to catch unmatched paths

* Add custom 404 error message for invalid URLs
(cherry picked from commit 946871c)

Co-authored-by: Yeonguk Choo <choo121600@gmail.com>
ephraimbuddy pushed a commit that referenced this pull request Dec 3, 2025
… URLs (#58546) (#58629)

* Add error handling route to catch unmatched paths

* Add custom 404 error message for invalid URLs
(cherry picked from commit 946871c)

Co-authored-by: Yeonguk Choo <choo121600@gmail.com>
Copilot AI pushed a commit to jason810496/airflow that referenced this pull request Dec 5, 2025
…e#58546)

* Add error handling route to catch unmatched paths

* Add custom 404 error message for invalid URLs
itayweb pushed a commit to itayweb/airflow that referenced this pull request Dec 6, 2025
…e#58546)

* Add error handling route to catch unmatched paths

* Add custom 404 error message for invalid URLs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:translations area:UI Related to UI/UX. For Frontend Developers. backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch translation:default

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants