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

GH-111: Add Privacy Manifest in Kronos #112

Merged

Conversation

didix21
Copy link
Contributor

@didix21 didix21 commented Mar 20, 2024

  • Add PrivacyInfo.xcprivacy to Kronos.xcoproj.
  • Add in the Privacy Manifest UserDefaults API with reason CA92.1.
  • Include PrivacyInfo.xcprivacy file in the Package.swift.
  • Upgrade minimum version of swift-tools-version to accept resources in Targets.
  • Include PrivacyInfo.xcprivacy file in Pods file.

- Add PrivacyInfo.xcprivacy to Kronos.xcoproj.
- Add in the Privacy Manifest UserDefaults API with reason CA92.1.
- Include PRivacyInfo.xcprivacy file in the Package.swift.
- Upgrade minimum version of swift-tools-version to accept resources in Targets.
- Include PrivacyInfo.xcprivacy file in Pods file.

Signed-off-by: Dídac Coll <[email protected]>
@didix21 didix21 force-pushed the maintenance/iOS-2725-privacy-info branch from dc06b4c to df7b966 Compare March 20, 2024 11:22
@sberrevoets
Copy link
Member

It's mostly (ex) Lyft employees looking after this library—privacy manifests are on our radar so please stay tuned!

@didix21
Copy link
Contributor Author

didix21 commented Apr 15, 2024

any updates on this?

@sberrevoets
Copy link
Member

Thanks for submitting! This looks good, the test failures here are obviously unrelated so working toward getting that fixed and seeing if we should just merge this in the meantime.

@tinder-chriswebb
Copy link

That's great news! We're eager to see the next release of Kronos with Privacy Manifest compliance, as Apple's new policies will take effect on May 1. Is a codeowner able to merge this?

@keith keith merged commit 94f6c11 into MobileNativeFoundation:main Apr 22, 2024
2 of 5 checks passed
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.

5 participants