-
Notifications
You must be signed in to change notification settings - Fork 50
Syw UID2-3502 fix android doco #581
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
sunnywu
commented
May 24, 2024
- add more info (R8/Proguard warning) from android sdk repo readme into IMA/GMA plugin guide,
- change minimum requirements for different uses of SDK and make it clearer to app publishers
… and change minimum requirements for different uses of SDK and make it clearer to app publishers
| - [Gradle](#installing-with-gradle) | ||
|
|
||
| - [ Maven](#installing-with-maven) | ||
| - [Maven](#installing-with-maven) |
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.
previous typo
… and change minimum requirements for different uses of SDK and make it clearer to app publishers
… and change minimum requirements for different uses of SDK and make it clearer to app publishers
IanDBird
left a comment
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.
Looks great, thanks!
docs/sdks/uid2-sdk-ref-android.md
Outdated
| - Minimum target Android version: 4.4+ / API 19+ (SDK) 5.0+ | ||
|
|
||
|
|
||
| To run the development app as mentioned in the [Client-Side Integration Guide for Mobile](../guides/integration-mobile-client-side#client-side-integration-example) or build binary from source code, minimum requirements are as follows: |
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.
Good clarification, thanks!
genwhittTTD
left a comment
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.
Few copy mods. Your call. LMK if any questions.
|
|
||
| 1. Check out the main branch of the [UID2 SDK for Android source code repository on GitHub](https://github.com/IABTechLab/uid2-android-sdk/tree/main). | ||
| 1. In Android Studio (Jellyfish/v2023.3.1 or whichever future version supports the Android Gradle Plugin version required by the UID2 SDK for Android release at the time), open the directory that you checked out. | ||
| 1. In Android Studio (check the version required in the [Minimum Requirements section](../sdks/uid2-sdk-ref-android.md#minimum-requirements) in UID2 SDK for Android Reference Guide, open the directory that you checked out. |
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.
- In Android Studio (check the version required in the Minimum Requirements section in the UID2 SDK for Android Reference Guide), open the directory that you checked out.
(close parens missing + "section" isn't part of the link and can go outside link text + added "the")
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.
fixed
| - [SDK](https://central.sonatype.com/artifact/com.uid2/uid2-android-sdk) | ||
| - [UID2 SDK for Android Reference Guide](../sdks/uid2-sdk-ref-android.md) | ||
| 1. [UID2 Android GMA Plugin v0.5.0](https://central.sonatype.com/artifact/com.uid2/uid2-android-sdk-gma/) | ||
| 1. If you are using R8 or Proguard, add the necessary file mentioned in [here](#notes-for-using-r8-or-proguard) |
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.
- If you are using R8 or Proguard, add the applicable file specified in Notes for Using R8 or ProGuard
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.
fixed
| </dependency> | ||
| ``` | ||
|
|
||
| ## Notes for using R8 or ProGuard |
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.
Notes for Using R8 or ProGuard
(capitalization)
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.
fixed
|
|
||
| ## Notes for using R8 or ProGuard | ||
|
|
||
| If you are using R8 the shrinking and obfuscation rules are included automatically. |
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.
If you are using R8, the shrinking and obfuscation rules are included automatically.
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.
fixed
|
|
||
| If you are using R8 the shrinking and obfuscation rules are included automatically. | ||
|
|
||
| ProGuard users must manually add the options from |
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.
If you are using ProGuard, you must manually add the option specified in the following file:
uid2-gma.pro.
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.
fixed
| - [SDK](https://central.sonatype.com/artifact/com.uid2/uid2-android-sdk) | ||
| - [UID2 SDK for Android Reference Guide](../sdks/uid2-sdk-ref-android.md) | ||
| 1. [UID2 IMA Plugin for Android v0.5.0](https://central.sonatype.com/artifact/com.uid2/uid2-android-sdk-ima) | ||
| 1. If you are using R8 or Proguard, add the necessary file mentioned in [here](#notes-for-using-r8-or-proguard) |
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.
- If you are using R8 or Proguard, add the applicable file specified in Notes for Using R8 or ProGuard
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.
fixed
|
|
||
| ## Notes for using R8 or ProGuard | ||
|
|
||
| If you are using R8 the shrinking and obfuscation rules are included automatically. |
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.
If you are using R8, the shrinking and obfuscation rules are included automatically.
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.
fixed
| If you are using R8 the shrinking and obfuscation rules are included automatically. | ||
|
|
||
| ProGuard users must manually add the options from | ||
| [uid2-ima.pro](https://github.com/IABTechLab/uid2-android-sdk/blob/main/securesignals-ima/uid2-ima.pro). |
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.
If you are using ProGuard, you must manually add the option specified in the following file: uid2-ima.pro.
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.
fixed
docs/sdks/uid2-sdk-ref-android.md
Outdated
| - Minimum target Android version: 4.4+ / API 19+ (SDK) 5.0+ | ||
|
|
||
|
|
||
| To run the development app as mentioned in the [Client-Side Integration Guide for Mobile](../guides/integration-mobile-client-side#client-side-integration-example) or build binary from source code, minimum requirements are as follows: |
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.
To run the development app (see Client-Side Integration Guide for Mobile), or to build the binary from source code, the minimum requirements are as follows:
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.
fixed
docs/sdks/uid2-sdk-ref-android.md
Outdated
|
|
||
| To run the development app as mentioned in the [Client-Side Integration Guide for Mobile](../guides/integration-mobile-client-side#client-side-integration-example) or build binary from source code, minimum requirements are as follows: | ||
|
|
||
| - Android Studio version: Check the Android Gradle Plugin (AGP) version required by the UID2 SDK specified in the [code repository](https://github.com/IABTechLab/uid2-android-sdk/blob/main/gradle/libs.versions.toml), and check [here](https://developer.android.com/build/releases/gradle-plugin) for the corresponding Android Studio version required |
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.
- Android Studio version: Check the Android Gradle Plugin (AGP) version required by the UID2 SDK specified in the code repository, and check the Android Gradle release notes for the corresponding Android Studio version required.
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.
fixed