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

TimeTable: Add auto-refresh and trip ID tracking #427

Merged
merged 2 commits into from
Dec 4, 2024

Conversation

ksharma-xyz
Copy link
Owner

@ksharma-xyz ksharma-xyz commented Dec 4, 2024

TL;DR

Added trip ID tracking and auto-refresh functionality for real-time journey updates in the trip planner.

Update tripID logic and save past journey in a map, so that we can still see it as long as the user is on the screen, even though new data can keep coming in and add to the list.

What changed?

  • Added TransportationProperties data class to capture tripCode and realtimeTripId
  • Modified journey ID generation to use leg-specific identifiers
  • Implemented trip tracking using a mutable map to store and update journey information
  • Added auto-refresh mechanism to periodically update journey times and information
  • Enhanced time text updates for journey cards to show accurate timing information

Screenshots

Fix.-.Update.Mins.ago.time.properly.mov

Why make this change?

To improve real-time tracking of journeys and provide more accurate, up-to-date information to users. The addition of trip IDs and auto-refresh functionality ensures that users always see the most current journey status and timing information.

@ksharma-xyz ksharma-xyz changed the title works with flicker TimeTable: Add auto-refresh and trip ID tracking Dec 4, 2024
Copy link
Owner Author

ksharma-xyz commented Dec 4, 2024

@ksharma-xyz ksharma-xyz marked this pull request as ready for review December 4, 2024 13:09
@ksharma-xyz ksharma-xyz added this to the Release v1.0 iOS + Android milestone Dec 4, 2024
… be called twice

1. When screen is open - call periodically after REFRESH_TIME_TEXT_DURATION
2. When new data comes from api every AUTO_REFRESH_TIME_TABLE_DURATION
@ksharma-xyz ksharma-xyz merged commit 19b0aed into main Dec 4, 2024
2 checks passed
Copy link
Owner Author

Merge activity

  • Dec 4, 8:36 AM EST: A user merged this pull request with Graphite.

@ksharma-xyz ksharma-xyz deleted the 12-04-works_with_flicker branch December 4, 2024 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant