Alerts page bus/ferry/access tab: remove current/planned filter#2939
Merged
Alerts page bus/ferry/access tab: remove current/planned filter#2939
Conversation
…The page defaults to All, which is the desired effect. This component is still used on the Alerts tab of the schedules page for modes other than subway and commuter rail.
…am on the alerts page)
…ally, so it should work this time
… elements that have been removed. It now looks for the Access button instead.
joshlarson
approved these changes
Feb 12, 2026
Co-authored-by: Josh Larson <[email protected]>
…ub.com/mbta/dotcom into lev/a11y/alerts_remove_current-planned
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Scope
Asana Ticket: Alerts page bus/ferry/access tab: remove current/planned filter
Implementation
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
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