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

GPS: updates #18669

Merged
merged 3 commits into from
Jan 10, 2024
Merged

GPS: updates #18669

merged 3 commits into from
Jan 10, 2024

Conversation

Florin9doi
Copy link
Contributor

@Florin9doi Florin9doi commented Jan 6, 2024

  • Simulate fresh updates to satisfy MAPLUS 1/2 when real GPS data isn't available
  • Remove NmeaListener workaround for Android 29 SDK
  • Add constants for GPGGA tokens

@Florin9doi
Copy link
Contributor Author

MAPLUS apps also need GPS location from within Japan, but this is likely expected.

@Florin9doi Florin9doi changed the title GPS: Simulate fresh updates to satisfy MAPLUS 1/2 when real GPS data isn't available GPS: updates Jan 7, 2024
@hrydgard hrydgard added this to the v1.17.0 milestone Jan 7, 2024
Comment on lines -78 to -80
// Use reflection to work around a bug in the Android 29 SDK.
// https://stackoverflow.com/questions/57975969/accessing-nmea-on-android-api-level-24-when-compiled-for-target-api-level-29
try {
Copy link
Owner

Choose a reason for hiding this comment

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

Hm, how come we can remove this?

Copy link
Owner

Choose a reason for hiding this comment

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

Oh wait, never mind, apparently it was indeed just the SDK that was buggy, not Android. it's ok.

@hrydgard hrydgard merged commit 5b4ed75 into hrydgard:master Jan 10, 2024
18 checks passed
@Florin9doi Florin9doi deleted the gps_auto_update_time branch January 10, 2024 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants