Skip to content

Implement Advertising ID service and integrate ATT flow in AdsProvider#59

Merged
JakubMrozek merged 3 commits intodevelopfrom
duc/advertising-ids
Feb 17, 2026
Merged

Implement Advertising ID service and integrate ATT flow in AdsProvider#59
JakubMrozek merged 3 commits intodevelopfrom
duc/advertising-ids

Conversation

@phiviet01
Copy link
Collaborator

@phiviet01 phiviet01 commented Feb 11, 2026

  • Added an internal AdvertisingIdService to resolve IDFV (iOS) and IDFA/GAID automatically for /preload.
  • /preload uses AdvertisingIdService IDs, with AdsProvider values as fallback.
  • Added startup ATT flow from AdsProvider: runs once per app process, only on iOS 14.5+, only when status is notDetermined.
  • Added tests.

@phiviet01 phiviet01 marked this pull request as ready for review February 11, 2026 20:27
Copy link
Member

@JakubMrozek JakubMrozek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We discussed it, and it would be great to rewrite and use native module instead of external packages.

@JakubMrozek JakubMrozek marked this pull request as draft February 12, 2026 12:21
@JakubMrozek JakubMrozek changed the title Implement Advertising ID service and integrate ATT flow in AdsProvider Implement Advertising ID service and integrate ATT flow in AdsProvider [Don't merge] Feb 12, 2026
@phiviet01 phiviet01 marked this pull request as ready for review February 16, 2026 21:42
@JakubMrozek JakubMrozek changed the title Implement Advertising ID service and integrate ATT flow in AdsProvider [Don't merge] Implement Advertising ID service and integrate ATT flow in AdsProvider Feb 17, 2026
@JakubMrozek JakubMrozek merged commit b8c8301 into develop Feb 17, 2026
1 check passed
@JakubMrozek JakubMrozek deleted the duc/advertising-ids branch February 17, 2026 11:59
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.

2 participants

Comments