This should be a squash merge, the PR title should serve for release as conventional commit message
Checklist
I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] Yes
My change supports the following platforms;
[ ] Android
[x] 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
[x] No
Test Plan
The base of the PR style-wise is adding the platform to package.json then using it in podspecs, that is tested already by library consumer so should work
The compile ifdefs will be tested on iOS at least and that should be sufficient
Think react-native-firebase is great? Please consider supporting the project with any of the below:
Description
This supercedes:
8109
It is based on that PR from @adamkoch but I can't push to that branch and I have done a series of little fixups on it
Related issues
Discussion:
Release Summary
This should be a squash merge, the PR title should serve for release as conventional commit message
Checklist
Android
iOS
e2e
tests added or updated inpackages/\*\*/e2e
jest
tests added or updated inpackages/\*\*/__tests__
Test Plan
The base of the PR style-wise is adding the platform to package.json then using it in podspecs, that is tested already by library consumer so should work
The compile ifdefs will be tested on iOS at least and that should be sufficient
Think
react-native-firebase
is great? Please consider supporting the project with any of the below:React Native Firebase
andInvertase
on Twitter