Skip to content

Add PrivacyInfo.xcprivacy #301

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dp221125
Copy link

@dp221125 dp221125 commented Mar 15, 2024

This PR adds a privacy manfiest (PrivacyInfo.xcprivacy), Apple has not been mentioned here, but this SDK is using the User Defaults API

Additionally, since the swift-tools-version in Package.swift is set to 5.0, it was not possible to add resources to the target. Functions that include resources available from version 5.3, so I have updated it to 5.3. (swift-package-manager github)

resolve #299 #300

@CH3COOH
Copy link

CH3COOH commented Mar 21, 2024

@dp221125 Thank you for your pull request! Since NSPrivacyCollectedDataTypes is not included in PrivacyInfo, will there be no error when generating a Privacy Report?

@dp221125
Copy link
Author

@dp221125 Thank you for your pull request! Since NSPrivacyCollectedDataTypes is not included in PrivacyInfo, will there be no error when generating a Privacy Report?

I'm sorry for the late reply. I have fixed it. 4b59143

@dp221125
Copy link
Author

@sunshinejr could you review PR, please?

@applebest
Copy link

@sunshinejr请您查看一下 PR 好吗?

@sunshinejr Please merge pr

@@ -1,4 +1,4 @@
// swift-tools-version:5.0
// swift-tools-version:5.3

Choose a reason for hiding this comment

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

Having this set at version 5.0 could be important for some projects. The way to make sure it can still work for those people would be to create a [email protected] file which has the original 5.0 tools version.

Anyone requiring that version would be well used to having to manually handle the privacy manifest for their dependencies. And anyone using a version 5.3 or higher would get this Package.swift package description

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.

Privacy manifest required by Apple
4 participants