-
Notifications
You must be signed in to change notification settings - Fork 986
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: add PrivacyInfo.xcprivacy for CordovaLib & app template #1383
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1383 +/- ##
==========================================
- Coverage 78.25% 78.24% -0.02%
==========================================
Files 15 15
Lines 1789 1788 -1
==========================================
- Hits 1400 1399 -1
Misses 389 389 ☔ View full report in Codecov by Sentry. |
c53419b
to
8d69afd
Compare
Should we need to add 'Cordova' directory like this?
|
8d69afd
to
b655d18
Compare
Good catch. I've added some changes to clean up the CordovaLib project structure a little bit (both for Xcode and for the Swift Package) and the files should be there as expected now. |
b655d18
to
151aee0
Compare
61c5adb
to
4ebe09a
Compare
Note to self: https://github.com/apache/cordova-coho/pull/324/files needs to be updated after this is merged to use the correct path to Info.plist for versioning |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Hi. Are there any plans to back port this to Cordova iOS 6.x? Our plans are to update to 7.x in the fall. |
@mg Sadly no. Cordova is maintained by volunteers and we just don't have the resources or capacity to support multiple active branches. |
@dpogue roger, thanks for the response |
This reverts commit 1974dbb. Revert "feat: add privacy-manifest config support (apache#1406)" This reverts commit c97845a. Revert "ci(gh-action): add Apache RAT & package license checker workflow w/ license header additions (apache#1408)" This reverts commit d316558. Revert "fix: use PROVISIONING_PROFILE_SPECIFIER for manual codesigning (apache#1405)" This reverts commit af6335e. Revert "feat: add PrivacyInfo.xcprivacy for CordovaLib & app template (apache#1383)" This reverts commit b400b70. Revert "fix: WASM MIME type error by specifying it in Info.plist template (apache#1374)" This reverts commit 902df96. Revert "chore: update package & package-lock (apache#1404)" This reverts commit 2091208. Revert "chore: bump 7.1.0-dev for next minor release (apache#1403)" This reverts commit 766adcf. Revert "chore(deps-dev): bump @babel/traverse from 7.21.4 to 7.23.2 (apache#1382)" This reverts commit 92017bb. Revert "chore: Update Slack signup link in SUPPORT_QUESTION.md (apache#1380)" This reverts commit be4c884. Revert "chore: bump version 7.0.2-dev" This reverts commit a1f3ace.
Platforms affected
iOS
Motivation and Context
Starting in April 2024, Apple requires that newly submitted apps include a Privacy Manifest that declares whether the app collects personal data, whether it uses that data for tracking purposes, and whether the app uses privacy-impacting system APIs.
Description
Adds a PrivacyInfo.xcprivacy manifest file to CordovaLib, declaring that CordovaLib does not do any tracking or use any privacy-impacting APIs.
Adds a PrivacyInfo.xcprivacy manifest file to the generated app template, with empty declarations. It is up to the app author to provide the correct privacy details for their application.
Minor cleanups to the CordovaLib project to improve both Xcode subproject and Swift package use.
Testing
Tested locally in Xcode.
Checklist