geofire-js icon indicating copy to clipboard operation
geofire-js copied to clipboard

Firebase 6.x.x incompatibility and firebase vs firebase-admin clash.

Open ReverseControl opened this issue 4 years ago • 6 comments

Feature request: can we please get a geofire version compatible with firebase 6.x.x?

Version info

UNMET PEER DEPENDENCY [email protected] ├── [email protected] ├── [email protected] ├── [email protected] ├── [email protected] ├── [email protected] ├── [email protected] ├── [email protected] ├── [email protected]

npm ERR! peer dep missing: firebase@^2.4.0 || 3.x.x || 4.x.x || 5.x.x, required by [email protected]

Expected behavior

Compatible geofire library with the latest firebase.

Actual behavior

The requirement restriction makes it impossible to use "firebase" and 'firebase-admin" together as it shows the known issue where google cloud complains that a "database" has already been registered when these two packages are imported in the same file.

ReverseControl avatar Sep 11 '19 07:09 ReverseControl

ping

ziyaddin avatar Oct 13 '20 15:10 ziyaddin

@ziyaddin can are you having the exactly the same issue as @ReverseControl or something different?

I ask as the OP was asking a year ago.

I'm using:

`firebase": "^7.23.0`
`geofire": "^5.0.1`

I am using React/React Native, I've had some issues myself but making some progress, replying in case I can offer some suggestions but would need more info.

prideaux avatar Oct 13 '20 20:10 prideaux

@prideaux thanks for reply. Mine is almost the same. I just get warning instead of error.

image

ziyaddin avatar Oct 15 '20 20:10 ziyaddin

Is there any workaround for this issue?

I was able to develop and test without any issues locally, but I'm not able to deploy 😢

mschuetz72 avatar Nov 14 '20 18:11 mschuetz72

@mschuetz72 what is the error you are facing with during deployment?

ziyaddin avatar Nov 14 '20 19:11 ziyaddin

Hey @ziyaddin. Please disregard my comment about the failing deployment. The message was Deployment error. Function failed on loading usercode. and after going back to the last working commit and applying change by change it turned out that geofire wasn't referenced as dependency in the package file 🤷‍♂️ All working now, sorry for the confusion...

mschuetz72 avatar Nov 15 '20 09:11 mschuetz72