SDK-2223-DMA-Consent-Params-Implementation

BranchMetrics / android-branch-deep-linking-attribution

The Branch Android SDK for deep linking and attribution. Branch helps mobile apps grow with deep links / deeplinks that power paid acquisition and re-engagement campaigns, referral programs, content sharing, deep linked emails, smart banners, custom user onboarding, and more.

MIT License

399 stars 156 forks source link

Reference

SDK-2223 -- Design and implement key/values to represent consent param https://branch.atlassian.net/browse/SDK-2223

Description

Added following API in Branch Class. public void setDMAParamsForEEA(boolean eeaRegion, boolean adPersonalizationConsent, boolean adUserDataUsageConsent) ;
Branch SDK will save DMA Compliance Parameters, set by developer using above API, and will send to server with install, open and event requests.
Added unit test for testing above API.

Testing Instructions

All unit tests should pass.
Call API setDMAParamsForEEA and check -- SDK is sending value of eeaRegion, with every /v1/install , /v1/open, /v2/event , v1/pageview request. -- SDK is sending value of adPersonalizationConsent / adUserDataUsageConsent with every /v1/install , /v1/open, /v2/event , v1/pageview request only if -
- eeaRegion is true OR
- eeaRegion is false and adPersonalizationConsent / adUserDataUsageConsent is also set to false Ref : https://www.notion.so/brancheng/Google-DMA-Compliance-c19ed28d2ea74b2b9fa4d2d77f8cdcbe

Risk Assessment [`HIGH` || `MEDIUM` || `LOW`]

LOW

[ X] I, the PR creator, have tested — integration, unit, or otherwise — this code.

cc @BranchMetrics/saas-sdk-devs for visibility.

Codecov Report

Attention: 14 lines in your changes are missing coverage. Please review.

Comparison is base (5d55d8d) 37.56% compared to head (3029b75) 37.74%.

Files	Patch %	Lines
...rc/main/java/io/branch/referral/ServerRequest.java	22.22%	13 Missing and 1 partial :warning:

Files

Patch %

Lines

...rc/main/java/io/branch/referral/ServerRequest.java

22.22%

13 Missing and 1 partial :warning:

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #1167 +/- ## ============================================ + Coverage 37.56% 37.74% +0.18% - Complexity 660 675 +15 ============================================ Files 59 59 Lines 6115 6154 +39 Branches 918 921 +3 ============================================ + Hits 2297 2323 +26 - Misses 3375 3387 +12 - Partials 443 444 +1 ```

BranchMetrics / android-branch-deep-linking-attribution