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

Cannot add location when device's location mode is device only #764

Closed
sjl872964789 opened this issue Jun 8, 2020 · 7 comments
Closed
Assignees
Labels
Projects
Milestone

Comments

@sjl872964789
Copy link

Describe the bug
The location cannot be added, and there are no explanations and reminders for the user. After investigation, I found that the problem is caused by the "device only" mode. Omni notes should prompt the user to switch the location mode just as it prompts the user to open the location. Otherwise, the user may not know why the location cannot be added

Context

  • Device: galaxy a6s
  • OS version: Android 8.1.0
  • App version: 6.0.5

How to reproduce
Steps to reproduce the behavior:

  1. Change location mode to "device only"
  2. Install Omni Notes and open
  3. Click on plus button in the lower right corner
  4. Click on "Text note'
  5. Click the paper clip button
  6. Click on "Location'
  7. See "LOCATION NOT FOUND"

Expected behavior
The user should be prompted that the current positioning mode cannot be located and needs to switch to high accuracy mode

Screenshots
Recorded video see attachment
record.zip

@federicoiosue
Copy link
Owner

First of all thanks for the great bug reporting!

I'll dig further into asap, meanwhile, when you say "device only" it means that you're using only GPS isn't it?

@federicoiosue federicoiosue self-assigned this Jun 11, 2020
@sjl872964789
Copy link
Author

Thank you for your kind reply. Yes, "device only" means that I'm using only GPS. There are two ways to make your device in "device only" location mode. The first way is to turn on location setting and turn off the network. The second way is to select "device only" mode through a special setting, which is available in my mobile phone(galaxy a6s-android8.1.0) and the Android simulator I often use (pixel2-android8.0). At this time, no matter whether your network is turned on or not, you can only use GPS for positioning. The way to turn on this setting is as follows:
ezgif com-video-to-gif

@tingsu
Copy link

tingsu commented Jul 5, 2020

Simiar issue here. Any plan to resolve this problem? Thanks.

@federicoiosue federicoiosue added this to the 6.1.0 milestone Jul 15, 2020
@sachin-0102
Copy link
Contributor

Can I work on this issue?

@federicoiosue
Copy link
Owner

@sachin-0102 of course you can. Please give a look to README to follow code contribution guidelines!

@sachin-0102
Copy link
Contributor

I've made a PR on this issue...Check it out and let me know if something needs improvement

@federicoiosue
Copy link
Owner

Fixed by #778 and 9388e24

@federicoiosue federicoiosue added this to To do in Omni Notes via automation Oct 1, 2020
@federicoiosue federicoiosue moved this from To do to In progress in Omni Notes Oct 3, 2020
@federicoiosue federicoiosue moved this from In progress to Done in Omni Notes Oct 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Omni Notes
  
Done
Development

No branches or pull requests

4 participants