Skip to content

[Symphony]: Symphony logic final (final) version - #3433

Merged
lvachon1 merged 2 commits into
mainfrom
lev/change/stale_alert_logic_aug26
Aug 24, 2026
Merged

[Symphony]: Symphony logic final (final) version#3433
lvachon1 merged 2 commits into
mainfrom
lev/change/stale_alert_logic_aug26

Conversation

@lvachon1

@lvachon1 lvachon1 commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Scope

Asana Ticket: 🐬 Symphony logic final (final) version

Implementation

Changed the business logic for stale? alerts, now they're only for the symphony work. Updated alerts_test tests to suit the new logic.

Screenshots

Symphony alert hidden on Subway Status

Screenshot 2026-08-19 at 11 44 37 AM Screenshot 2026-08-19 at 11 46 54 AM

Symphony alert hidden on Green-E line

Screenshot 2026-08-19 at 11 44 12 AM

Symphony alert shown in Green-E alerts

Screenshot 2026-08-19 at 11 43 58 AM

Symphony alert hidden in Green line

Screenshot 2026-08-19 at 11 43 27 AM

Symphony alert shown in Green alerts

Screenshot 2026-08-19 at 11 43 40 AM

How to test

http://localhost:4001/ & http://localhost:4001/alerts/subway - Confirm the symphony alert isn't shown in Subway Status
http://localhost:4001/schedules/Green-E/line & http://localhost:4001/schedules/Green/line - Confirm the alert doesn't show on the line diagram but does on the Alerts tab

Depending on the order in which these PRs are merged you may want to test the departures page as well.

http://localhost:4001/departures?route_id=Green&direction_id=1&stop_id=place-symcl - Confirm the alert shows
http://localhost:4001/departures?route_id=Green&direction_id=1&stop_id=place-hymnl - Confirm the alert doesn't show

…e symphony work. Updated alerts_test tests to suit the new logic. There will likely be more
@lvachon1
lvachon1 requested a review from a team as a code owner August 19, 2026 15:49
@lvachon1
lvachon1 requested a review from joshlarson August 19, 2026 15:49

@joshlarson joshlarson left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good! I appreciate that this changes the definition of stale? without changing anything about its interaction with the outside world, which means that it's fully, 100% independent of your other stale-related PR 🎉


I feel compelled to flag that technically, this would match an alert that was created to close Symphony and another stop, so if, say, a single alert closed both Symphony and Prudential, then that alert would also get marked as stale, and wouldn't show up.

I still consider this non-blocking, both because I think the scenario above is exceedingly unlikely, and also surprisingly difficult to solve for (since individual platforms are typically informed entities during station closures as well, so it's not as simple as saying "are all of the informed stop entities place-symcl?"), but I also think it is worth noting... data can do surprising things!

Comment thread lib/alerts/alert.ex Outdated
@lvachon1
lvachon1 merged commit 82a9a12 into main Aug 24, 2026
20 checks passed
@lvachon1
lvachon1 deleted the lev/change/stale_alert_logic_aug26 branch August 24, 2026 14:30
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.

2 participants