Repository navigation
Conversation
Fairmount Line riders travelling within Zone 1A who pay with a CharlieCard can transfer for free, within 2 hours of their first tap, to subway, bus, or Silver Line. Previously the trip planner charged these Commuter Rail legs separately (e.g. Red Line + Fairmount Zone 1A was $4.80 instead of $2.40). Fairmount Line legs that start and end in Zone 1A now participate in the 2-hour transfer window; all other Commuter Rail legs are still charged separately. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WWCAcgKBaiwFgiFBHjr48c
Keeps the original add_fare_from_leg/2 clause structure, widening the transfer-eligible clause's guard instead of introducing a separate predicate function. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WWCAcgKBaiwFgiFBHjr48c
Unavailable trips come from the `mbta-ma-us-initial` feed and still display a fare, so the Fairmount Zone 1A check should match route IDs from either feed. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WWCAcgKBaiwFgiFBHjr48c
Collaborator
|
Thanks for contributing! Our team's a bit backlogged at the moment, but I'll aim to review before end of week. |
joshlarson
approved these changes
Nov 13, 2026
joshlarson
left a comment
joshlarson
left a comment
Collaborator
There was a problem hiding this comment.
This is great! Thanks for fixing this!
joshlarson
enabled auto-merge (squash)
November 13, 2026 22:15
joshlarson
added
Not Ready for Translation
This branch has not been deployed
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.
Fairmount Line riders travelling within Zone 1A who pay with a CharlieCard can transfer for free, within 2 hours of their first tap, to subway, bus, or Silver Line. Previously the trip planner charged these Commuter Rail legs separately (e.g. Red Line + Fairmount Zone 1A was $4.80 instead of $2.40).
Scope
Currently, the trip planner doesn't reflect the free transfers between Fairmount Line Zone 1A and the rest of the subway/bus/ferry. This PR fixes that.
Implementation
fairmount_zone_1atrips are now treated like route types[0, 1, 3, 4]for transfer purposes.🤖 This PR's code was generated by Claude, though I read over the changes.
Screenshots