RadarCOVID / radar-covid-android

Native Android app using DP^3T Android sdk to handle Exposure Notifications API from Google
Mozilla Public License 2.0
382 stars 84 forks source link

SDK 30 Location error notification #44

Closed ponytojas closed 3 years ago

ponytojas commented 3 years ago

Starting in Android 11, localization is not necessary for the exposures api to work, as you can read in the following link.

https://blog.google/inside-google/company-announcements/update-exposure-notifications/

On Android 11, which will soon be released, users will be able to use Exposure Notification apps without turning on the device location setting.

The App keeps sending an error notification if you try to use it without using the location (Even if bluetooth is on)

I attach a screenshot of the notification and the android version used IMAGE 2020-10-25 18:38:32 IMAGE 2020-10-25 18:38:30

It's a minor problem, but every now and then the notification keeps jumping and from what I can see it's something related to the app and not so much the API, although I may be wrong.

Thanks for your efforts and such a well done app

wiyarmir commented 3 years ago

It's fixed in release 1.0.5 of the DP-3T SDK https://github.com/DP-3T/dp3t-sdk-android/releases/tag/1.0.5

ponytojas commented 3 years ago

Yes support is added since the release you're pointing, but I'm only talking about the notification.

What I'm talking in this issue is the fact that even if the app is working properly without location, there's still a notification asking for it

wiyarmir commented 3 years ago

The notification is triggered by the SDK. Once they upgrade the SDK, it'll go away 〰️ .

ponytojas commented 3 years ago

The this issue can be closed. Thanks for the explanation