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

feat: add GAID fetching and saving to google_ad_id #93

Merged
merged 9 commits into from
Jan 10, 2025

Conversation

comfrt1k
Copy link
Contributor

@comfrt1k comfrt1k self-assigned this Dec 24, 2024
@comfrt1k
Copy link
Contributor Author

  1. В первом пункте задания сказано, что надо запрашивать пермишны:

Для запроса свойства нужно запросить пермишны. Это должна быть опциональная конструкция, которую разработчик клиента может добавить в инициализацию SDK, чтобы мы могли получать этот ID и отправлять к себе.

На оф сайте написано это:

Кроме того, при обновлении приложений до Android 13 или более поздней версии необходимо указать в файле манифеста обычное разрешение Google Play следующим образом:
<uses-permission android:name="com.google.android.gms.permission.AD_ID"/>

И это:

Некоторые SDK, например Google Mobile Ads SDK (play-services-ads), могут уже объявлять это разрешение в манифесте библиотеки SDK. Если ваше приложение использует эти SDK в качестве зависимостей, разрешение AD_ID из манифеста библиотеки SDK будет по умолчанию объединено с основным манифестом вашего приложения, даже если вы явно не объявляете это разрешение в основном манифесте вашего приложения.

То есть, в рантайме запрашивать ничего не надо.
Вышесказанное также решает проблему, описанную во втором пункте:

При этом магазин может уже установить этот запрос прав без нашего участия (в крупных магазинах этот кейс будет в 100% случаев), поэтому наш SDK не должен конфликтовать с этой настройкой, но должен уметь забирать AD_ID.

В таком случае манифесты просто объединятся, а ad_id будет нам доступен

  1. GAID получается при каждой инициализации sdk, тк получать актуальный можно только так (пользователь может его менять и удалять)

  2. Ну и пруфы на скринах ниже:

Если пользователь не удалил gaid или не запретил его использование:
image

Если gaid удален :
image

@comfrt1k comfrt1k marked this pull request as ready for review December 25, 2024 11:31
@TorinAsakura TorinAsakura removed their request for review December 25, 2024 11:58
@comfrt1k comfrt1k requested a review from looee1q December 25, 2024 12:02
@comfrt1k comfrt1k requested a review from looee1q December 26, 2024 12:07
@comfrt1k comfrt1k requested a review from looee1q December 27, 2024 07:39
looee1q
looee1q previously approved these changes Jan 9, 2025
@looee1q looee1q requested a review from TorinAsakura January 9, 2025 16:05
@TorinAsakura TorinAsakura merged commit d3cd1c3 into master Jan 10, 2025
1 check passed
@TorinAsakura TorinAsakura deleted the feat/add-gaid-support branch January 10, 2025 16:13
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.

3 participants