redux-beacon
redux-beacon copied to clipboard
Add allowIDFA to react native GA
Checklist
- [ ] I have added tests that prove my fix is effective or that my feature works.
- [x] I have added all necessary documentation (if appropriate)
What was done
Added functionality to allow IDFA collection for the react-native Google Analytics package. This signals to GA whether the app is allowed to use the advertiser identifier for the device to deliver adds or collect ad click attribution.
Not sure how to go about adding a test for this, we've been using this change in production for a while now.
Nice work. I'm curious about something though...are there scenarios where we'd need to set allowIDFA
multiple times throughout the lifecycle of the app? I'm thinking that perhaps this is more of a setting rather than an event:
function GoogleAnalytics(
trackingId: string,
GoogleAnalyticsTracker: any
settings?: { allowIdfa?: boolean }
)
I agree with you - the only counter point would be to keep consistency with react-native-google-analytics-bridge
?
Hmm...I see...
Honestly, I'd prefer if we remain consistent with our own abstractions. Otherwise all our targets might diverge in methodology/usage. I'd like the experience from target to target to be as similar as possible.
@ttmarek not a problem! I agree with you.