Skip to content

Conversation

@joshlarson
Copy link
Contributor

Scope

Asana Ticket: ❗[AM Alerts] Don't show train(s) specific alerts on line diagram

Implementation

Screenshots

Screenshot 2026-02-02 at 3 28 08 PM

(for context, the Wellesley Farms alert applied only to a few trips, while the Auburndale one applied to the whole day - see screenshot 👇)
Screenshot 2026-02-02 at 4 07 27 PM

How to test

Create a station closure or station bypass test alert for a commuter rail route, and have it apply to some specific trips (rather than all trips). Examine the line diagram, and note that the stations you closed don't show as Skipped.

Create a station closure or station bypass test alert for a commuter rail route, and have it apply to all trips. Examine the line diagram, and note that the stations you closed do appear as Skipped.

@joshlarson joshlarson requested a review from a team as a code owner February 2, 2026 21:10
@joshlarson joshlarson requested a review from thecristen February 2, 2026 21:10
@joshlarson joshlarson force-pushed the jdl/fix/dont-show-train-specific-alerts-on-line-diagram branch from 9f5c6bd to 8887127 Compare February 2, 2026 21:15
Copy link
Collaborator

@thecristen thecristen left a comment

Choose a reason for hiding this comment

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

👍🏼

@joshlarson joshlarson enabled auto-merge (squash) February 3, 2026 20:28
@joshlarson joshlarson merged commit 8b6e2b4 into main Feb 3, 2026
17 checks passed
@joshlarson joshlarson deleted the jdl/fix/dont-show-train-specific-alerts-on-line-diagram branch February 3, 2026 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants