-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
fix(analytics, ios): handle RNFirebaseAnalyticsWithoutAdIdSupport == false Podfile case correctly #6078
Conversation
This pull request is being automatically deployed with Vercel (learn more). react-native-firebase – ./🔍 Inspect: https://vercel.com/invertase/react-native-firebase/C3eyXzECQuc57RMHqDWFfEpZ6qi1 react-native-firebase-next – ./website_modular🔍 Inspect: https://vercel.com/invertase/react-native-firebase-next/7iuq1CSuJLnVunYE7oRREv4zwyVC [Deployment for 8ac2cf0 canceled] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh my, that looks like a 🤦 error / incorrect assumption.
Of course most that define this will only define it if they want to set it to true, but, also (of course) you can define it false if you like.
Thanks for the catch + PR!
Codecov Report
@@ Coverage Diff @@
## main #6078 +/- ##
=============================================
+ Coverage 52.92% 72.11% +19.20%
=============================================
Files 208 109 -99
Lines 10220 4618 -5602
Branches 1626 1040 -586
=============================================
- Hits 5408 3330 -2078
+ Misses 4558 1210 -3348
+ Partials 254 78 -176 |
The PR Title failure is okay, it's more of a warning for maintainers on merge since github handles single-commits in a surprising way. If the rest are green, we're ✔️ |
Sorry for the delay publishing this one! Oversight on my part I did not realize we had this fix unpublished. I just kicked off the release process now |
Description
Right know it looks only if it is defined and not if it set as true, same error as in older appsflyer-react-native-plugin xD
Release Summary
Fix RNFirebaseAnalyticsWithoutAdIdSupport flag in Podfile
Checklist
Android
iOS
e2e
tests added or updated inpackages/\*\*/e2e
jest
tests added or updated inpackages/\*\*/__tests__