-
Notifications
You must be signed in to change notification settings - Fork 5.9k
[action] NEW ACTION for App Data Usages: upload_app_data_usage_to_app_store #17592
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
Conversation
Generated by 🚫 Danger |
Thank you!! 🥇 |
Are you going to release that :)? |
@joshdholtz Apple is making a change where this will be required on Dec 8th, 2020. |
@ryechus Yup yup! Finishing this PR up today 😊 |
1cf7a89
to
c4ac030
Compare
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.
Mostly LGTM, just have a few minor comments and some questions
fastlane/lib/fastlane/actions/upload_app_data_usage_to_app_store.rb
Outdated
Show resolved
Hide resolved
fastlane/lib/fastlane/actions/upload_app_data_usage_to_app_store.rb
Outdated
Show resolved
Hide resolved
fastlane/lib/fastlane/actions/upload_app_data_usage_to_app_store.rb
Outdated
Show resolved
Hide resolved
fastlane/lib/fastlane/actions/upload_app_data_usage_to_app_store.rb
Outdated
Show resolved
Hide resolved
fastlane/lib/fastlane/actions/upload_app_data_usage_to_app_store.rb
Outdated
Show resolved
Hide resolved
fastlane/lib/fastlane/actions/upload_app_data_usage_to_app_store.rb
Outdated
Show resolved
Hide resolved
spaceship/lib/spaceship/connect_api/models/app_data_usage_category.rb
Outdated
Show resolved
Hide resolved
spaceship/lib/spaceship/connect_api/models/app_data_usage_data_protection.rb
Outdated
Show resolved
Hide resolved
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.
🚀
I just marked some minor things which looked like typos to me
fastlane/lib/fastlane/actions/upload_app_privacy_details_to_app_store.rb
Outdated
Show resolved
Hide resolved
fastlane/lib/fastlane/actions/upload_app_privacy_details_to_app_store.rb
Outdated
Show resolved
Hide resolved
fastlane/lib/fastlane/actions/upload_app_privacy_details_to_app_store.rb
Outdated
Show resolved
Hide resolved
fastlane/lib/fastlane/actions/upload_app_privacy_details_to_app_store.rb
Show resolved
Hide resolved
fastlane/lib/fastlane/actions/upload_app_privacy_details_to_app_store.rb
Outdated
Show resolved
Hide resolved
…p_store.rb Co-authored-by: Manu Wallner <manu@supermil.ch>
…p_store.rb Co-authored-by: Manu Wallner <manu@supermil.ch>
…p_store.rb Co-authored-by: Manu Wallner <manu@supermil.ch>
…p_store.rb Co-authored-by: Manu Wallner <manu@supermil.ch>
Hey @joshdholtz 👋 Thank you for your contribution to fastlane and congrats on getting this pull request merged 🎉 Please let us know if this change requires an immediate release by adding a comment here 👍 |
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.
Congratulations! 🎉 This was released as part of fastlane 2.170.0 🚀
@joshdholtz Thank you so much! In case anyone else is confused, if your app isn't collecting data for a category you don't need to include the dictionary in the json file. |
i have it working for non-Admin 🎉 we currently flag it in our system if we completed the app privacy for a non-Admin, the only place i've seen an error during deliver, which is a bit further along the process than we'd like. |
Don't know if it's linked with @sonjz comment but in our case, we did completed this informations directly in App Store Connect some weeks ago but we get the error |
@guyaumetremblay that doesn't sound related. is it possible on the App Privacy page that one of your categories was missed or Publish at the top wasn't hit? For my issue, received a thoughtful comment about |
@sonjz OMG we didn't see the Publish button at the top!! Thanks a lot!! |
FYI and for anyone curious, we started a proposal for implementing a new feature on CocoaPods that would hopefully work nicely with this fastlane feature here, as it would generate the json file used here by this fastlane action… from all the info extracted from the libs/pods you use in your app 🙂 (feedbacks on the proposal welcome) |
Motivation
App support for updating App Data Usage for new app privacy policies
Description
Adds a new action called
upload_app_privacy_details_to_app_store
. Not only does this new action upload App Data Usage to App Store Connect API, it will provide interactive questions (similar to what the App Store Connect dashboard does). The answers from the interactive questions will upload the App Data Usage to App Store Connect but also create a JSON file that can be put into source control and used again without needing to answer the questions.Interactive questions
With JSON from previously answered questions
JSON File
The questions are tedious to answer. The JSON file is not straight forward to manually put together since it can have a lot of options. This action generating and reusing the JSON file should help make this process much easier for the user while also keeping a history of what was answered in source control 😊
Categories
PAYMENT_INFORMATION
,CREDIT_AND_FRAUD
,OTHER_FINANCIAL_INFO
,PRECISE_LOCATION
,SENSITIVE_INFO
,PHYSICAL_ADDRESS
,EMAIL_ADDRESS
,NAME
,PHONE_NUMBER
,OTHER_CONTACT_INFO
,CONTACTS
,EMAILS_OR_TEXT_MESSAGES
,PHOTOS_OR_VIDEOS
,AUDIO
,GAMEPLAY_CONTENT
,USTOMER_SUPPORT
,OTHER_USER_CONTENT
,BROWSING_HISTORY
,SEARCH_HISTORY
,USER_ID
,DEVICE_ID
,PURCHASE_HISTORY
,PRODUCT_INTERACTION
,ADVERTISING_DATA
,OTHER_USAGE_DATA
,CRASH_DATA
,PERFORMANCE_DATA
,OTHER_DIAGNOSTIC_DATA
,OTHER_DATA
,HEALTH
,FITNESS
,COARSE_LOCATION
Purposes
THIRD_PARTY_ADVERTISING
,DEVELOPERS_ADVERTISING
,ANALYTICS
,PRODUCT_PERSONALIZATION
,APP_FUNCTIONALITY
,OTHER_PURPOSES
Data Protections
Use
DATA_LINKED_TO_YOU
orDATA_NOT_LINKED_TO_YOU
Optionally add
DATA_USED_TO_TRACK_YOU
Example: Not Collecting Data
Example: Collecting Data
Run through CLI
With interactive questions
With JSON file output from first step
Run with Fastfile
With interactive questions
With JSON file output from first step