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

Split Privacy Manifest by modules #482

Merged
merged 5 commits into from
May 18, 2024
Merged

Split Privacy Manifest by modules #482

merged 5 commits into from
May 18, 2024

Conversation

GLinnik21
Copy link
Collaborator

No description provided.

<false/>
<key>NSPrivacyTrackingDomains</key>
<array/>
<key>NSPrivacyCollectedDataTypes</key>
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Even though this module transmits crash and diagnostic data, I decided not to redeclare it here, as crash data types would present in aggregated report from modules it depends on.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Getting rid of legacy as 5.1 doesn't support resources and 5.7 is the oldest version that can be reasonably supported at the moment.

Copy link
Collaborator

@bamx23 bamx23 left a comment

Choose a reason for hiding this comment

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

Looks nice and clean, thanks!

Package.swift Outdated Show resolved Hide resolved
Package.swift Outdated Show resolved Hide resolved
@GLinnik21 GLinnik21 mentioned this pull request May 18, 2024
@GLinnik21 GLinnik21 changed the title Split privacy report Split Privacy Manifest by modules May 18, 2024
@GLinnik21 GLinnik21 merged commit c7218ad into master May 18, 2024
19 checks passed
@GLinnik21 GLinnik21 deleted the split-privacy-report branch May 18, 2024 14:29
@GLinnik21 GLinnik21 added this to the 2.0.0 milestone Sep 20, 2024
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.

2 participants