[tvOS] How to config for react-native-tvOS?

[nhannfq]

I'm trying build a tvOS application for the Apple TV, react-native-firebase doesn't seem to have support forfor tvOS, �does it?

[mikehardy]

I would be shocked if it worked. It's on my wishlist to provide that but I haven't had time.
I believe the work to do, if you've got the time, is mostly going in and adding the appropriate ifdefs/ifndefs etc that toggle out apple API symbols that are (or are not) defined for tvOS, along with appropriate markup in associated index.d.ts files (that are processed into our API documentation) noting where things are not supported on tvOS, perhaps with a main docs page in the core/app module about tvOS specifically that links to the upstream firestore-ios-sdk package documentation on what modules work for tvOS and what don't, and what the differences are from the iOS platform

[mikehardy]

That alone would be a very positive step and I think cover 99% of potential future failures.

Separately - it appears that Detox may be made to work on tvOS as the previous was that underlying framework EarlyGrey did not work on tvOS, but then that changed and subsequently that framework was dropped anyway such that an in-repo DetoxSync framework is used.

As such, getting Detox to work on tvOS may be a possibliity now (and...just maybe...another one of those ifdef'ing festivals that is somehow easier then expected - could happen!).

Or it could be that on-device testing is better done in some other framework, preferably one that does not rely on the JSI (our current Detox/Jet system uses JSI...) and preferably one that works on all platforms (where I define that as android/ios/web/macOS/catalyst/windows - everything...). Not sure what that other platform would be but maybe some sort of webdriver appium thing - I only mention this because perhaps you have some domain knowledge here and could share wisdom, doesn't hurt to ask :-)

But concretely: just a build check? Still very useful + totally acceptable

[grigdodon]

any update on this?
when trying to build, I get:

Framework not found FirebaseAnalytics
Linker command failed with exit code 1 (use -v to see invocation)

[mikehardy]

Hey @grigdodon 👋 - nope, no concrete steps though I'm still open to any reasonable PR to get tvOS working, I continue to think it would just be a matter of ifdefing things out and then a build check that copies our current ios e2e yml and does everything except the actual e2e run.

When I see errors like yours though I get worried - I wonder if that's related to using unsupported modular_headers Podfile workarounds that will never work for react-native-firebase, instead of using use_frameworks! :linkage => :static ? If not that, then perhaps analytics isn't supported for tvOS, I haven't looked at the specific list of modules that work or don't work in a while

[dev-folks]

Hey!
I also spent a lot of time trying to find a successful case of using firebase for react-native-tvos. It seems that the user request for this is negligible - too few questions and information...

react-native-tvos/react-native-tvos#811

[mikehardy]

I just took a look at this myself as I'd like to see react-native-tvos supported (and react-native-web, supported well for that matter vs just as shim-hacks like I've done)

I do not currently have a way to test tvOS support (though I just did another look at the 2 open feature requests in react-native-test-app and tried to confirm some assumptions there about adding it).

If you have a way to test current react-native-tvos or are willing to spend the time to make a skeleton repository (that I could just drop a google services info plist/json set into) then we could get it building I'm sure. In other repositories I manage, it's just a matter of doing the right compiler ifdefs to exclude API usage that is invalid on tvOS platform

[douglowder]

I'll take a look to see if the underlying Firebase Cocoapods dependencies actually support tvOS. If so, then we can probably do this.

[adamkoch]

#8109

[douglowder]

Thanks @adamkoch I'll have a look

[mikehardy]

#8111 has what I think is final merge-worthy state on this - I just extended the style you posted, Adam, to all supported firebase components, and switched the auth ifdef compile guards to ios-only to match upstream - pretty simple really, happy for such a small change to enable an entire platform

[adamkoch]

Amazing, thank you!

[dev-folks]

Oh yes, that's amazing, you are legends!