Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nfc/clipper: #3991

Merged
merged 3 commits into from
Dec 2, 2024
Merged

nfc/clipper: #3991

merged 3 commits into from
Dec 2, 2024

Conversation

ted-logan
Copy link
Contributor

@ted-logan ted-logan commented Nov 6, 2024

What's new

  • Correct the station id for BART station San Leandro
  • Fill in the station id for BART station Bay Fair
  • Add MUNI station ids for the MUNI's T line "Central Subway" project, which opened three new underground stations in 2022

Verification

  • To verify this requires specific rides on San Francisco Bay Area transit with a Clipper card. Maybe @tomholford has been to some of these stations recently since we're now tagging each other in our pull requests :)

  • MUNI records only the entering station, and doesn't require tagging off to exit the system or complete a trip. Here's a screenshot viewing my Clipper card record of tagging into Chinatown - Rose Pak Station:

    flipper-chinatown

    Before my change, this station showed up as "Unknown (001a)"

  • Tagging into Union Square / Market Street Station:

    flipper-union-square

  • Tagging into Yerba Buena/Moscone Station

    flipper-yerba-buena

  • BART records both the entry and exit stations. Here's a screenshot viewing my Clipper card record of a trip from Bay Fair north one stop to San Leandro:

    flipper-bay-fair-san-leandro

    Before my change, the San Leandro station id was incorrect so the trip was incorrectly reported as starting at San Leandro and ending at "Unknown (0020)".

Checklist (For Reviewer)

  • PR has description of feature/bug or link to Confluence/Jira task
  • Description contains actions to verify feature/bugfix
  • I've built this code, uploaded it to the device and verified feature/bugfix

Update the BART station ids for the San Lorenzo and Bay Fair stations in
the East Bay of the San Francisco Bay Area.
Add MUNI station ids for the T line "Central Subway" project, which
opened three new underground stations in 2022.
@tomholford
Copy link
Contributor

tomholford commented Nov 8, 2024

@ted-logan Nice work.

Maybe @tomholford has been to some of these stations recently since we're now tagging each other in our pull requests :)

I checked the card, unfortunately there are no records for those BART or Muni stops. Nonetheless, based on both #3937 and #3969, these changes LGTM.

@skotopes skotopes merged commit 3d6fd9b into flipperdevices:dev Dec 2, 2024
11 checks passed
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