-
Notifications
You must be signed in to change notification settings - Fork 883
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
getDoc() is never resolved nor rejected using Remix and Firestore Lite #8195
Comments
Hi @jdgamble555 Could you please try to use lite SDK directly without Remix and check if you are running into same problem? |
It is only a Remix problem, as it works fine in NextJS. But the Remix problem only happens with J |
Hi @jdgamble555 , It is likely the issue is under |
Following the linked issue in Remix, it looks like it is in fact a Remix issue (trying an experimental flag in Remix fixes this issue). I'll close this for now, notify us to reopen it if there turns out to be an error on our end. |
Operating System
Windows 11, Vercel Production
Browser Version
Chrome 124.0.6367.61
Firebase SDK Version
10.11.0
Firebase SDK Product:
Firestore
Describe your project's tooling
Remix does not resolve on regular
getDoc
path. I am using the latest version of Remix, although there is a rollup bug so I had to override the latest version in package.json.Describe the problem
Everything works find and resolves as expected when using
firebase/firestore
, but when switching tofirebase/firestore/lite
, the path does not resolve. I believe it is related to the same issue here.Steps and code to reproduce issue
Create a new Remix project and with a path
about
. Fetch a document using the loader andfirebase/firestore
. Make sure that works. Now switch tofirebase/firestore/lite
and you will see it never resolves.Debug Info
Here is the test repo (need to add any Firebase credentials):
https://github.com/jdgamble555/remix-firebase-test/blob/master/app/lib/about.server.ts
https://github.com/jdgamble555/remix-firebase-test
The test deployed version (deployed working without Lite):
https://remix-firebase-test.vercel.app/
J
The text was updated successfully, but these errors were encountered: