-
-
Notifications
You must be signed in to change notification settings - Fork 458
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
Add Consent Mode v2 (Android & iOS) #878
Conversation
…ULT_ALLOW_ANALYTICS_STORAGE, GOOGLE_ANALYTICS_DEFAULT_ALLOW_AD_STORAGE, GOOGLE_ANALYTICS_DEFAULT_ALLOW_AD_USER_DATA, GOOGLE_ANALYTICS_DEFAULT_ALLOW_AD_PERSONALIZATION_SIGNALS preferences to both android & ios
…lse to switch case
Add Consent Mode v2 (Android & iOS)
Updated iOS sdk to 10.24.0; Bumped up Android sdks versions
I've also updated iOS sdk to 10.24.0 and bumped up Android sdks versions |
Thanks, I'm going to try to make some time next week to merge this and some other PRs, fix some niggling bugs and get a release out |
Downgraded GoogleSignIn to 6.2.4
Hey, thanks for your contribution. Can you please check if my modifications are correct? Because I don't really know Objective-c and I had to rely on your previous code, to add the new features. Also I had to downgrade GoogleSignIn back to 6.2.4, because v7.0.0+ requires some code migration and I don't know exactly what modifications I need to do in the FirebasePlugin.m, to get this to work |
PR Type
What kind of change does this PR introduce?
PR Checklist
Please check your PR fulfills the following requirements:
Bugfixes:
New features/enhancements:
What is the purpose of this PR?
GA4 now requires consent (EEA)
https://support.google.com/analytics/answer/14275483
Does this PR introduce a breaking change?
What testing has been done on the changes in the PR?
Forked the firebasex test project and tested the consent functionality
https://github.com/pinguluk/cordova-plugin-firebasex-test
What testing has been done on existing functionality?
Other information