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

feat(ios): Add bundled resources for privacy manifest #275

Closed
wants to merge 2 commits into from

Conversation

breautek
Copy link
Contributor

Platforms affected

ios

Motivation and Context

Support Apple's Privacy Manifest requirements

Description

  • Created a resource bundle
  • Added PrivacyInfo.xcprivacy to the resource bundle
  • Load bundle into iOS project with resource-file in plugin.xml

The bundle is prefixed with Cordova's internal CDV namespace.

Note for third-party plugins: When creating a bundle or even class names, it is recommended to establish your own namespace and utilize a unique identifier to prevent conflicts with other plugins.

Testing

  • platform add
  • plugin add
  • archive project
  • generated privacy report

Checklist

  • I've run the tests to see all new and existing tests pass
  • I added automated test coverage as appropriate for this change
  • Commit is prefixed with (platform) if this change only applies to one platform (e.g. (android))
  • If this Pull Request resolves an issue, I linked to the issue in the text above (and used the correct keyword to close issues using keywords)
  • I've updated the documentation if necessary

With the better understanding of what Apple's definition of "Collect"

“Collect” refers to transmitting data off the device in a way that allows you and/or your third-party partners to access it for a period longer than what is necessary to service the transmitted request in real time.

Ref: https://developer.apple.com/app-store/app-privacy-details/
@breautek
Copy link
Contributor Author

Because after learning that collected means basically moving data off, there's no point of adding a privacy manifest I think, though the readme could still apply. I'm going to create a new PR for cleanliness.

@breautek breautek closed this Apr 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant