Skip to content

Adds NSPrivacyAccessedAPICategoryUserDefaults to privacy manifest #243

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

Merged
merged 2 commits into from
Apr 8, 2025

Conversation

kkostov
Copy link
Contributor

@kkostov kkostov commented Mar 29, 2025

Hi,

Once we updated one of our apps to the latest version of the SDK, it got rejected by App Store Review for it was missing the NSPrivacyAccessedAPICategoryUserDefaults entry in the privacy manifest. We tracked it down to the TelemetryDeck SDK.

This PR addresses the issue by adding the missing description to both privacy manifests. Our app was accepted using a fork of the library with this change 🤩 .

Cheers,
K

PS: perhaps a step can be added to the GitHub workflow to check that? e.g. https://github.com/crasowas/app_store_required_privacy_manifest_analyser

@kkostov kkostov requested review from winsmith and Jeehut March 29, 2025 11:04
Copy link
Contributor

@Jeehut Jeehut left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kkostov Thank you for this! Looks good to me. Merging.

@Jeehut Jeehut merged commit 0cb66ec into main Apr 8, 2025
6 checks passed
@Jeehut Jeehut deleted the fix/privacy-manifest branch April 8, 2025 09:04
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