Skip to content
This repository has been archived by the owner on Jun 24, 2022. It is now read-only.

Remove text about location permissions #1019

Closed
wants to merge 1 commit into from

Conversation

voronaam
Copy link

@voronaam voronaam commented Aug 6, 2020

Remove Android only text that explains to user a perculiar situation with the location permissions, which is no longer true.

The application does NOT request the ACCESS_COARSE_LOCATION permission anymore.

This is in line with the API documentation: https://developers.google.com/android/exposure-notifications/exposure-notifications-api

Hence the block no longer applies.

Reviewer checklist

This is a suggested checklist of questions reviewers might ask during their review:

  • Does this meet a user need?
  • Is it accessible?
  • Is it translated between both offical languages?
  • Is the code maintainable?
  • Have you tested it?
  • Are there automated tests?
  • Does this cause automated test coverage to drop?
  • Does this break existing functionality?
  • Should this be split into smaller PRs to decrease change risk?
  • Does this change the privacy policy?
  • Does this introduce any security concerns?
  • Does this significantly alter performance?
  • What is the risk level of using added dependencies?
  • Should any documentation be updated as a result of this? (i.e. README setup, etc.)

@voronaam voronaam requested a review from a team as a code owner August 6, 2020 23:20
@timarney
Copy link
Member

timarney commented Aug 9, 2020

Adding some reference links here:

Current Android

The Exposure Notification technology uses Bluetooth scanning to understand what devices are near one another. On all phones running Android 6.0 and above, to use Bluetooth scanning, the device location setting needs to be turned on ...

https://support.google.com/android/answer/9930236?hl=en

Android 11

On Android 11, which will soon be released, users will be able to use Exposure Notification apps without turning on the device location setting. We’re making this update for Exposure Notifications only, given that ENS has been designed in such a way that neither the system nor the apps using it can infer device location through Bluetooth scanning, and apps that are allowed to use ENS are subject to additional policies that disallow automatic collection of location. All other apps and services will still be prohibited from performing Bluetooth scanning unless the device location setting is on.

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

@timarney
Copy link
Member

Closing this based on #1019 (comment)

@timarney timarney closed this Aug 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants