Skip to content
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

Closed
jdgamble555 opened this issue Apr 22, 2024 · 4 comments
Closed

Comments

@jdgamble555
Copy link

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.

"overrides": {
  "rollup": "4.15.0"
}

Describe the problem

Everything works find and resolves as expected when using firebase/firestore, but when switching to firebase/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 and firebase/firestore. Make sure that works. Now switch to firebase/firestore/lite and you will see it never resolves.

Debug Info

[2024-04-22T13:05:29.729Z]  @firebase/firestore: Firestore (10.11.0_lite): RestConnection Sending RPC 'BatchGetDocuments' 0x23ee4115: https://firestore.googleapis.com/v1/projects/test-projects-19046/databases/(default)/documents:batchGet {
  documents: [
    'projects/test-projects-19046/databases/(default)/documents/about/ZlNJrKd6LcATycPRmBPA'
  ]
}

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

@jdgamble555 jdgamble555 added new A new issue that hasn't be categoirzed as question, bug or feature request question labels Apr 22, 2024
@jbalidiong jbalidiong added needs-attention and removed new A new issue that hasn't be categoirzed as question, bug or feature request labels Apr 22, 2024
@cherylEnkidu
Copy link
Contributor

Hi @jdgamble555

Could you please try to use lite SDK directly without Remix and check if you are running into same problem?

@jdgamble555
Copy link
Author

It is only a Remix problem, as it works fine in NextJS. But the Remix problem only happens with firebase/firestore/lite. Not sure if there is something in the package that would cause this?

J

@cherylEnkidu
Copy link
Contributor

Hi @jdgamble555 ,

It is likely the issue is under Remix instead of Firestore Lite. The Firestore SDK team doesn't have context for Remix. Could you please consult Remix team for help?

@hsubox76
Copy link
Contributor

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.

@firebase firebase locked and limited conversation to collaborators May 26, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants