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

Incorrect message regarding location #6512

Open
srujner opened this issue Nov 15, 2024 · 1 comment
Open

Incorrect message regarding location #6512

srujner opened this issue Nov 15, 2024 · 1 comment

Comments

@srujner
Copy link

srujner commented Nov 15, 2024

ODK Collect version

2024.3.0 Beta 8

Android version

Android 14,15

Device used

Samsung Galaxy M23, Pixel 6a

Problem description

During regression testing, I came across a test in which there were several required fields to check. https://sd-odk.atlassian.net/browse/NODK-1333 And I happened to have Location disabled on my phone, but Precise Location was enabled which caused me confusion when I saw an error dialog that I need to turn on the Precise Location permission.

Steps to reproduce the problem

  1. Scan QR code:
    Screenshot(187)

  2. Turn off Location in device setting

  3. Have Precise Location permission turn on for ODK Collect app

  4. Click on any map icon

  5. the dialog informing that Without the Precise Location permission Collect can't record your location.

Expected behavior

To be honest I'm not sure if this is an issue, but I wanted to discuss if there shouldn't be a different dialog when the whole Location is turned off.

Other information

XRecorder_15112024_102657.mp4
@seadowg
Copy link
Member

seadowg commented Nov 15, 2024

That's weird! It looks like from the code we expect to get told that the permission is granted, and then we check if location services enabled at the device level (and show a message if not). I'm surprised that we end up being told that the permission is "denied". I think this is something we could improve, but it's not high priority.

@seadowg seadowg moved this from inbox to later in ODK Collect Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: later
Development

No branches or pull requests

2 participants