-
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
feat: vertexAI package support #8236
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Hey guys! Appreciate your effort to enable vertex AI on React Native! Quick question, are you planning to use the native android/ios libs for vertex, or is this just leveraging the web js lib? I'm not seeing any native files in the PR's diff Thanks! |
Hey there @santitopo as you can see in the PR, it's currently implemented as a port of the web implementation from upstream, bent to match our local needs so that the auth and app-check integration (native) here are used, and with fetch polyfilled so that it can do streaming despite streams not being implemented in baseline react-native fetch. |
Hi, does anyone have an update on when this PR might be merged? Thanks! |
@tomasabreu going through internal review still but likely won't be long. Mostly commenting to say that as an open source repo it's always possible to clone the repo, checkout this branch, and run an |
Co-author attribution for this work needs to be added. When squashing + merging PR, we need to include the following in the commit body (i.e. text box that appears):
|
Description
Related issues
Release Summary
Checklist
Android
iOS
e2e
tests added or updated inpackages/\*\*/e2e
jest
tests added or updated inpackages/\*\*/__tests__
Test Plan
Think
react-native-firebase
is great? Please consider supporting the project with any of the below:React Native Firebase
andInvertase
on Twitter