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

Added privacy manifest info to PrivacyInfo files #320

Merged
merged 7 commits into from
Oct 11, 2024

Conversation

AimeeT8
Copy link
Contributor

@AimeeT8 AimeeT8 commented Oct 3, 2024

What it Does

  • Closes Create PrivacyInfo.xcprivacy file #310 Create PrivacyInfo.xcprivacy file
  • Description:
    User Defaults APIs with usage reason CA92.1.
    Name - linked to identity and app functionality.
    Email Address - linked to identity and app functionality.
    Phone number - linked to identity and app functionality.
    Photos or videos - linked to identity.
    User ID - linked to identity and app functionality.

How I Tested

NA

Notes

Used Privacymanifest.dev for XML template

@AimeeT8 AimeeT8 requested a review from mikaelacaron as a code owner October 3, 2024 17:58
Copy link
Owner

@mikaelacaron mikaelacaron left a comment

Choose a reason for hiding this comment

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

Great work!! I have several comments that need resolved please. If you have any questions, you can comment directly on that thread, otherwise click the resolve button once you've addressed my comment. Once you fix everything, please click the re-review button

Re-review button

Basic-Car-Maintenance.xcodeproj/project.pbxproj Outdated Show resolved Hide resolved
@@ -90,7 +90,6 @@
savedToolIdentifier = ""
useCustomWorkingDirectory = "NO"
debugDocumentVersioning = "YES"
askForAppToLaunch = "Yes"
Copy link
Owner

Choose a reason for hiding this comment

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

you may need to manually add this back, it shouldn't be removed

Copy link
Owner

Choose a reason for hiding this comment

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

easiest way is to open in VSCode and add this back manually
Also you can just push new commits, the PR will update automatically

@@ -8,5 +8,78 @@
Copyright (c) 2024 Icy App Studio LLC. All rights reserved.
-->
<plist version="1.0">
<dict/>
<dict>
Copy link
Owner

Choose a reason for hiding this comment

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

Can you list in the PR description what privacy things you checked (like which bits are included)

And please add that to the AppStoreListing.md documentation file (make a new section called "Privacy Info"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I was a little uncertain about adding a new section to the AppStoreListing.md file. After lots of googling I committed what I had done. If it's not right I can change it no problem.

@AimeeT8 AimeeT8 requested a review from mikaelacaron October 6, 2024 20:35
Copy link
Owner

@mikaelacaron mikaelacaron left a comment

Choose a reason for hiding this comment

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

Great work! I reformatted it a bit and wrote a comment of how to remove the development team in pbxproj

@@ -964,6 +964,7 @@
CODE_SIGN_STYLE = Automatic;
DEAD_CODE_STRIPPING = YES;
DEVELOPMENT_ASSET_PATHS = "\"Basic-Car-Maintenance/Preview Content\"";
DEVELOPMENT_TEAM = "";
Copy link
Owner

Choose a reason for hiding this comment

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

you've selected development team and set it to none, but the correct way to fix this, is to manually go into the pbxproj file in like VSCode and delete the line that gets created (because without it, this will set everyone's team to none, rather than using the xcconfig file like it should)

@mikaelacaron mikaelacaron merged commit 96e2ad9 into mikaelacaron:dev Oct 11, 2024
1 of 2 checks passed
@AimeeT8 AimeeT8 deleted the 310-privacy-info branch October 11, 2024 16:15
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.

Create PrivacyInfo.xcprivacy file
2 participants