Skip to content

[ferry] [fare]: Charlestown and Winthrop/Quincy fares changes#3272

Open
lvachon1 wants to merge 6 commits into
mainfrom
lev/feat/inner_harbor_fare
Open

[ferry] [fare]: Charlestown and Winthrop/Quincy fares changes#3272
lvachon1 wants to merge 6 commits into
mainfrom
lev/feat/inner_harbor_fare

Conversation

@lvachon1

Copy link
Copy Markdown
Contributor

Scope

Asana Ticket: ⛴️ 💸 Charlestown and Winthrop/Quincy fares changes

Implementation

Updated the Charlestown Ferry price
Updated the Winthrop/Quincy logic to detect long-way-around trips

Screenshots

Charlestown Ferry New Price

Screenshot 2026-06-17 at 1 27 04 PM

Winthrop/Quincy Ferry long-way-around via Winthrop

Screenshot 2026-06-17 at 1 05 26 PM

Winthrop/Quincy Ferry long-way-around via Quincy

Screenshot 2026-06-17 at 1 05 15 PM

Winthrop/Quincy Ferry Fan Pier to Logan

Screenshot 2026-06-17 at 1 05 01 PM

Winthrop/Quincy Ferry Fan Pier to Logan via Central Wharf

Screenshot 2026-06-17 at 1 04 47 PM

TODO:

What's with this?
Screenshot 2026-06-17 at 2 40 36 PM

It looks fine in the details of that trip, but looks like a bus in the summary.

How to test

http://localhost:4001/trip-planner

  • Confirm the Charlestown Ferry fare is now $2.40 (Note, you'll have to restart your server for the new fares to be used, normal auto-reload doesn't work.)
  • Confirm Winthrop/Quincy trips that don't leave the inner harbor cost $2.40
  • Confirm Winthrop/Quincy trips that visit either Winthrop or Quincy (regardless of ultimate destination or origin) cost $6.50

@lvachon1 lvachon1 requested a review from a team as a code owner June 17, 2026 18:42
@lvachon1 lvachon1 requested a review from jlucytan June 17, 2026 18:42
@lvachon1 lvachon1 marked this pull request as draft June 17, 2026 20:36
@lvachon1

Copy link
Copy Markdown
Contributor Author

Things seem to work, but the ferry test is failing on trip legs we don't offer. I'll have to look into either the fares logic or the test logic to make things cleaner.

@lvachon1 lvachon1 marked this pull request as ready for review June 22, 2026 14:51
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.

1 participant