fix(ios, crashlytics): allow Crashlytics inclusion w/o Analytics

[mikehardy]

Description

@react-native-firebase/crashlytics was still bringing in Firebase/Analytics pod on iOS which disqualifies apps from being in the kids category on the App Store

This came from the remaining Firebase/Core pod dep in RNFBCrashlytics.podspec, which is no longer necessary - Firebase/CoreOnly is sufficient and that dependency is satisfied transitively by RNFBApp.podspec

Related issues

#4072 - initial issue
#4131 - initial PR that fixed almost all the modules

Interested parties @pabloearg and @dlockwo

Release Summary

Title of PR

Checklist

• I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • Yes
• My change supports the following platforms;
  • Android
  • iOS
• My change includes tests;
  • e2e tests added or updated in packages/\*\*/e2e
  • jest tests added or updated in packages/\*\*/__tests__
• I have updated TypeScript types that are affected by my change.
• This is a breaking change;
  • Yes
  • No

Test Plan

I overhauled my demonstrator script so that it could run in a '--no-idfa' mode (excluding admob and analytics) to verify all modules except those two work correctly. Then inspection of the Podfile.lock showed we were free of all the forbidden (by Apple, for kids apps) dependencies

mikehardy/rnfbdemo@4a716e9

---

Think react-native-firebase is great? Please consider supporting the project with any of the below:

• 👉 Star this repo on GitHub ⭐️
• 👉 Follow React Native Firebase and Invertase on Twitter

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/invertase/react-native-firebase/kpaiqq7ux
✅ Preview: https://react-native-firebase-git-crashlytics-no-core-dep.invertase.vercel.app