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

App rejected by Google Play because "Data Safety Section: Location Data Type - Approximate Location" #5708

Closed
nicolas-raoul opened this issue Apr 29, 2024 · 9 comments

Comments

@nicolas-raoul
Copy link
Member

nicolas-raoul commented Apr 29, 2024

App Status: Rejected

Your app has been rejected and wasn't published due to the policy issue(s) listed below. If you submitted an update, the previous version of your app is still available on Google Play.

Issue found: Invalid Data safety form

We reviewed your app’s Data safety form in Play Console and found discrepancies between it and how the app collects and shares user data. All apps are required to complete an accurate Data safety form that discloses their data collection and sharing practices - this is required even if your app does not collect any user data.

We detected user data transmitted off device that you have not disclosed in your app’s Data safety form as user data collected.

You must ensure that your app’s Data safety section accurately reflects your app’s data collection, sharing, and handling practices. This includes data collected and handled through any third-party libraries or SDKs used in your app. When available, we’ve included details on SDKs that contain code similar to the code in your APK that may be sending user data off device. You can check if your app uses any of these SDKs, but note that this list of SDKs may not be exhaustive. You must review and account for all data collected and shared by your app.

Issue details

We found an issue in the following area(s):

  • Version code 1039: Policy Declaration - Data Safety Section: Location Data Type - Approximate Location

About the Data safety section in Google Play User Data policy

Your app must be in compliance with this policy. If your app continues to be non-compliant, your app updates will be rejected and your app may face additional enforcement actions in the future.

Please make changes to align your app’s Data safety form with the app’s behavior. This can be done by either:

  • Updating your form in Play Console to declare collection of Data Types noted below; or
  • Removing unwanted functionality and attributable code that collects this user data from your app or libraries used in your app, and when applicable to deactivate all non-compliant APKs.
    • To deactivate non-compliant APKS, you can create a new release and upload a compliant APK to each track containing the non-compliant APKs.
    • Be sure to increment the APK version code. If using staged rollout, be sure to set the release to 100% rollout.

For helpful resources, you can:

@nicolas-raoul
Copy link
Member Author

I think we collect location for Nearby, Explore>Map, location edition in the upload wizard and media details, and in the camera-based upload in certain conditions. I might be missing some.

As seen in the screenshot below, the app currently declares precise location as "Data collected" and "Data shared".

Maybe we don't explain sufficiently to the user before asking for permissions, or something like that? @sivaraam

Screenshot_2024-04-29-15-40-32-62_438047d178ae0bdcccdfb4df8486f35b

@sivaraam
Copy link
Member

Their explicit mention of "Approximate location" was peculiar. So I checked the data safety form and it indeed had a "Approximate location" field that we hadn't filled. So, I went ahead and filled the same.

I used the opportunity to make some more tweaks to the Data Safety form. Since we no longer share the location to Mapbox for analytics and also only share the location to Wikimedia Maps to obtain the tiles, I removed the "Analytics" category from Location and also mentioned that the location is not shared with third-parties.

I've republished the changes for review. Let's see how it goes

@sivaraam
Copy link
Member

sivaraam commented Apr 29, 2024

It just occurred to me that I had missed the location we gather via the image metadata and removed the changes sent to review for now. I had mentioned we only use the location ephemerally to process a request considering the "Nearby" scenario.

Does the location gathered via the metadata count as permanent storage of user location? What do you think? 🤔

@nicolas-raoul
Copy link
Member Author

Does the location gathered via the metadata count as permanent storage of user location?

I would say yes.

@sivaraam
Copy link
Member

sivaraam commented Apr 30, 2024

Ok, Nicolas. I've mentioned that the location data is collected permanently and submitted both the update and the new data saftey form for review.

Let's see how it goes.

PS: I noticed this interesting tool which claims that it helps with generating the data safety form for the app. Need to check if it could helps us in some way in the future 🙂

@sivaraam
Copy link
Member

sivaraam commented May 2, 2024

Ok, Nicolas. I've mentioned that the location data is collected permanently and submitted both the update and the new data saftey form for review.

Let's see how it goes.

The update went through this time. The v5.0.1 (#5528) has been published to testing now 🙂

@nicolas-raoul
Copy link
Member Author

Fantastic!!!

Please do not hesitate to update https://github.com/commons-app/commons-app-documentation/blob/master/android/Project-maintenance.md with your tips and any setting export available. :-)

@sivaraam
Copy link
Member

sivaraam commented May 3, 2024

Fantastic!!!

Please do not hesitate to update https://github.com/commons-app/commons-app-documentation/blob/master/android/Project-maintenance.md with your tips

Sure. I do have a few updates to do there which can be seen in a draft PR found below.

commons-app/commons-app-documentation#52

Will complete the changes while promoting the app to production and mark the PR as ready. 🙂

... and any setting export available. :-)

Do you suggest to export the answers to the data safty form and capture those in the document too?

@sivaraam
Copy link
Member

Closing this as I think we're done with this :-)

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

No branches or pull requests

2 participants