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

Describing use of required reason API #1076

Merged
merged 1 commit into from
Apr 8, 2024
Merged

Describing use of required reason API #1076

merged 1 commit into from
Apr 8, 2024

Conversation

GhenadieVP
Copy link
Contributor

@GhenadieVP GhenadieVP commented Apr 8, 2024

Description

We required to implement the Describing use of required reason API

From the documentation and our codebase, the only thing that requires to declare the reason is UserDefaults.
Even though the Apple email suggested that we do also need to provide the description for the following:

I didn't find exact usage for these in our codebase, and I am not sure how the analysis is made by Apple. Possible our third-party dependencies might use these, need to investigate them. I guess the only way to find out is to send the app again for review.

<dict>
<key>NSPrivacyAccessedAPITypeReasons</key>
<array>
<string>CA92.1</string>
Copy link
Contributor Author

@GhenadieVP GhenadieVP Apr 8, 2024

Choose a reason for hiding this comment

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

Value from User defaults APIs

@GhenadieVP GhenadieVP marked this pull request as ready for review April 8, 2024 08:59
Copy link
Contributor

@CyonAlexRDX CyonAlexRDX left a comment

Choose a reason for hiding this comment

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

LGTM

@GhenadieVP GhenadieVP merged commit 910a1db into main Apr 8, 2024
11 checks passed
@GhenadieVP GhenadieVP deleted the api_declaration branch April 8, 2024 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants