Skip to content

Conversation

@OS-pedrogustavobilro
Copy link
Contributor

@OS-pedrogustavobilro OS-pedrogustavobilro commented Nov 21, 2025

Description

Now the plugin has native timeout for watchPosition on Android -> By updating native android geolocation lib to latest version 2.1.0.

This aligns the behavior with getCurrentPosition on Android, and also with iOS (recently addressed in a separate PR)

Unlike in Capacitor plugin, we don't really have need to support interval parameter here, because the OutSystems plugin will have the same behavior as before; the difference being that for apps using this version of the plugin, the timeout is now handled on native side instead of in JavaScript wrapper layer.

Context

Type of changes

  • Fix (non-breaking change which fixes an issue)
  • Feature (non-breaking change which adds functionality)
  • Refactor (cosmetic changes)
  • Breaking change (change that would cause existing functionality to not work as expected)

Platforms affected

  • Android
  • iOS
  • JavaScript

Tests

Can use the "Location Sample App" for Android on both O11 and ODC.

Now if you perform a watchPosition and wait a number of seconds > timeout to grant location permission, it will no longer timeout instantly, because the timeout does not run on JS anymore, but rather natively after permission request.

By updating native android geolocation lib to latest version 2.1.0

Unlike in Capacitor plugin, we don't really have need to support `interval` parameter here, because the OutSystems plugin will have the same behavior as before.

References:

- https://outsystemsrd.atlassian.net/browse/RMET-4688
- ionic-team/ion-android-geolocation#8
@OS-pedrogustavobilro OS-pedrogustavobilro self-assigned this Nov 21, 2025
@OS-pedrogustavobilro OS-pedrogustavobilro merged commit e832649 into main Nov 21, 2025
@OS-pedrogustavobilro OS-pedrogustavobilro deleted the fix/RMET-4688/timeout-watch-android-native branch November 21, 2025 17:47
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.

4 participants