Update notes on non-default optional data collection #7632
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jerielng
force-pushed
the
ad-tracking-updates
branch
from
8609d07 to
4ba546f
Compare
hokstuff
approved these changes
Jul 3, 2024
_docs/_user_guide/data_and_analytics/user_data_collection/sdk_data_collection.md
Outdated
Show resolved
Hide resolved
…data_collection.md Co-authored-by: Daniel Hok <4797040+hokstuff@users.noreply.github.com>
bryanlogan
reviewed
Jul 8, 2024
| | ---------------- | ------------------ | ------------------------------------------------------ | --------------------------------------------------- | | ||
| | Device Ad Tracking Enabled | Android, iOS | [set(adTrackingEnabled:)](https://braze-inc.github.io/braze-swift-sdk/documentation/brazekit/braze/set(adtrackingenabled:)) for iOS; on Android, this is collected via [`Braze.setGoogleAdvertisingId()`](https://braze-inc.github.io/braze-android-sdk/kdoc/braze-android-sdk/com.braze/-i-braze/set-google-advertising-id.html) | This property requires additional app-level permissions, which must be granted by the integrator. | | ||
| | Device IDFA | iOS | Device identifier for advertisers | This requires the Ad Tracking Transparency framework, which will trigger additional privacy review from the App Store. For more details, see [set(identifierForAdvertiser:)](https://braze-inc.github.io/braze-swift-sdk/documentation/brazekit/braze/set(identifierforadvertiser:)) | | ||
| | Google Advertising ID | Android | Identifier for advertising within Google Play apps | This requires declaring explicit permissions in the app-level manifest file. For more details, refer to [Optional Google Advertising ID]({{site.baseurl}}/developer_guide/platform_integration_guides/android/initial_sdk_setup/optional_gaid_collection) |
There was a problem hiding this comment.
Maybe
"This requires the app to retrieve the GAID and pass it to Braze. For more details, refer to Optional Google Advertising ID"
We don't get the ID at all because code-scanners get tripped up by this. Also, we don't reference that permission anywhere on that page either. We just refer to the Google requirements.
_docs/_user_guide/data_and_analytics/user_data_collection/sdk_data_collection.md
Outdated
Show resolved
Hide resolved
jacksonemiller
approved these changes
Jul 8, 2024
Co-authored-by: Jackson Miller <71667842+jacksonemiller@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Updates the incorrect data collection details, specifically around ad tracking properties for both iOS and Android. These pieces of data are not collected by default on either Braze SDK and requires explicit customer opt-in.