PostHog / posthog-ios

PostHog iOS SDK
https://posthog.com/docs/libraries/ios
MIT License
32 stars 41 forks source link

Remove references to AdSupport framework? #4

Closed sjmadsen closed 3 years ago

sjmadsen commented 4 years ago

With Apple further restricting apps that track users in iOS 14, I'd like to see a version of this framework that removed any references to AdSupport or the IDFA (identifier for advertisers). The "upstream" repo that this one is based on has done this for their current 4.x release (see this pull request), allowing apps that want to collect the IDFA to provide it, rather than having the framework itself make those calls into AdSupport.

How do PostHog iOS users feel about such a change? I'm happy to make the changes and create a PR if there is interest.

jamesefhawkins commented 3 years ago

Hey @sjmadsen , sorry for the slow response here.

I think this makes total sense as a PR if it's something you'd use and then wouldn't mind sharing. I imagine functionality-wise we should have:

sjmadsen commented 3 years ago

I'll get a PR together in the next few days.

mariusandra commented 3 years ago

This PR is now merged and version 1.1.0 of the library is out! Thanks again @sjmadsen !