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

Privacy Manifest #448

Merged

Conversation

felipe-nipper
Copy link
Contributor

  • Added Privacy Manifest file as required by Apple

@brototyp
Copy link
Member

brototyp commented Apr 1, 2024

Thanks for the contribution! LGTM. I'll let the tests pass and merge

@brototyp brototyp merged commit dbc8b40 into matomo-org:develop Apr 1, 2024
6 of 7 checks passed
@brototyp
Copy link
Member

brototyp commented Apr 1, 2024

Merged and part of Version 7.6.0

@pau-lu
Copy link

pau-lu commented Apr 4, 2024

Hi @felipe-nipper - a quick question - does the manifest only disclose NSPrivacyAccessedAPITypes because NS PrivacyTracking, NSPrivacyTrackingDomains and NSPrivacyCollectedDataTypes are not applicable? The 2nd note here https://developer.apple.com/documentation/bundleresources/privacy_manifest_filesnote also says "You only need to supply NSPrivacyAccessedAPITypes for apps and 3rd party SDKs on iOS" but I think this means that NSPrivacyAccessedAPITypes dictionary is required in those SDKs in addition to the 3 above (to the extent they are applicable). Thank you.

@felipe-nipper
Copy link
Contributor Author

Hi @pau-lu, the values of NSPrivacyTracking, NSPrivacyTrackingDomains, and NSPrivacyCollectedDataTypes can change based on how you use the data. For instance, your usage of Matomo doesn't necessarily fit into the tracking requirements set by Apple. Hence, the values you're required to delineate can vary by project. Keeping this in mind, you should input this information at your app level and not in the third-party SDK. Additionally, it seems that NSPrivacyTrackingDomains and NSPrivacyCollectedDataTypes are not necessarily required.

@pau-lu
Copy link

pau-lu commented Apr 8, 2024

Hi @felipe-nipper - thank you!

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

Successfully merging this pull request may close these issues.

3 participants