Skip to content

feat: internal usage attribution ID#95

Merged
jokerttu merged 1 commit intomainfrom
feat/add_internal_usage_attribution_id
Apr 30, 2026
Merged

feat: internal usage attribution ID#95
jokerttu merged 1 commit intomainfrom
feat/add_internal_usage_attribution_id

Conversation

@jokerttu
Copy link
Copy Markdown
Contributor

@jokerttu jokerttu commented Apr 27, 2026 •
edited
Loading

  • Adds internal usage attribution ID

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I signed the CLA.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation
  • I added new tests to check the change I am making
  • All existing and new tests are passing.

caio1985
caio1985 requested changes Apr 27, 2026
View reviewed changes
Comment thread android/src/main/java/com/google/android/react/driversdk/SdkVersion.java Outdated
Comment thread ios/RNDriverSdkAttribution.m Outdated
@caio1985
Copy link
Copy Markdown
Member

caio1985 commented Apr 27, 2026

React Native Driver SDK:
Android: gmp_git_reactnativedriversdk_v0_android
iOS: gmp_git_reactnativedriversdk_v0_ios

@jokerttu jokerttu force-pushed the feat/add_internal_usage_attribution_id branch from 725f4ab to 8c8cb7b Compare April 30, 2026 07:42
@jokerttu jokerttu force-pushed the feat/add_internal_usage_attribution_id branch from 8c8cb7b to 85bfe25 Compare April 30, 2026 10:28
@jokerttu jokerttu requested a review from caio1985 April 30, 2026 10:46
@jokerttu jokerttu marked this pull request as ready for review April 30, 2026 10:46
@jokerttu jokerttu requested a review from illuminati1911 April 30, 2026 14:59
@jokerttu jokerttu merged commit b72e33a into main Apr 30, 2026
11 of 13 checks passed
@jokerttu jokerttu deleted the feat/add_internal_usage_attribution_id branch April 30, 2026 16:17
@googlemaps-bot googlemaps-bot mentioned this pull request Apr 30, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@caio1985 caio1985 caio1985 approved these changes

@ArturoSalazarB16 ArturoSalazarB16 Awaiting requested review from ArturoSalazarB16

@illuminati1911 illuminati1911 Awaiting requested review from illuminati1911

Assignees

@ArturoSalazarB16 ArturoSalazarB16

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jokerttu @caio1985 @ArturoSalazarB16