Skip to content

Alerts page bus/ferry/access tab: remove current/planned filter#2939

Merged
lvachon1 merged 17 commits intomainfrom
lev/a11y/alerts_remove_current-planned
Feb 12, 2026
Merged

Alerts page bus/ferry/access tab: remove current/planned filter#2939
lvachon1 merged 17 commits intomainfrom
lev/a11y/alerts_remove_current-planned

Conversation

@lvachon1
Copy link
Contributor

@lvachon1 lvachon1 commented Feb 11, 2026

Scope

Asana Ticket: Alerts page bus/ferry/access tab: remove current/planned filter

Implementation

  • Removed time range filter UI element from main Alerts (/alerts) page layout
  • Removed time_range logic from the Alerts page so time range URL params (?alerts_timerange=...) would be ignored
  • Updated unit test to reflect new logic (that ?alerts_timerange is not parsed).

We expect there to be some bookmarks with the old URLs. The URL parsing logic change is to make sure those links work and will receive the new behavior as well.

Screenshots

Screenshot 2026-02-11 at 2 13 35 PM Screenshot 2026-02-11 at 2 13 45 PM Screenshot 2026-02-11 at 2 14 04 PM

How to test

http://localhost:4001/alerts/ferry?alerts_timeframe=upcoming

-Observe that the All/Current/Planned filter UI is not present
-Observe that the URL param is ignored on this page
-Observe this also applies to other not-a-train alerts

http://localhost:4001/schedules/11/line (or some other bus with an alert)

-Observe that the filter UI is still present
-Observe that the URL parameters are parsed and still work
-Observe this also applies to other not-a-train schedule page alerts

Also currently live on Dev-Blue

@lvachon1 lvachon1 changed the title Lev/a11y/alerts remove current planned Alerts page bus/ferry/access tab: remove current/planned filter Feb 11, 2026
@lvachon1 lvachon1 marked this pull request as ready for review February 11, 2026 19:27
@lvachon1 lvachon1 requested a review from a team as a code owner February 11, 2026 19:27
@joshlarson joshlarson added the dev-blue Deploy to dev-blue label Feb 12, 2026
… elements that have been removed. It now looks for the Access button instead.
@lvachon1 lvachon1 merged commit 894ed08 into main Feb 12, 2026
22 checks passed
@lvachon1 lvachon1 deleted the lev/a11y/alerts_remove_current-planned branch February 12, 2026 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dev-blue Deploy to dev-blue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants