nuxt-community / firebase-module

🔥 Easily integrate Firebase into your Nuxt project. 🔥
https://firebase.nuxtjs.org
MIT License
642 stars 97 forks source link

Update peer dependency for firebase@9 #612

Closed shu-pf closed 2 years ago

shu-pf commented 2 years ago

The peer dependency in @nuxtjs/firebase is still set to firebase version 8, so please update it to 9.

Version

@nuxtjs/firebase: 8.1.1 firebase: 9.6.2 nuxt: 2.14.6

What is Expected?

Update peerDependencies

What is actually happening?

peerDependencies have not been updated.

bitpunk commented 2 years ago

"It is currently unclear when, and if, this module will support modular syntax (i.e. Firebase version 10+)." See #597 If you want to use firebase 9 and modular syntax you can checkout @lupas firebase 9 and nuxt.js article.

shu-pf commented 2 years ago

I read your link. It says that @nuxtjs/firebase v8 uses the compat syntax of Firebase SDK v9, so I thought, The peer dependency in the package.json of @nuxtjs/firebase v8 should be Firebase SDK v9.

lupas commented 2 years ago

@shu-pf is right, you can safely use v9 since we started using compat mode in v8.

I just released v8.2.2 fixing the peerDependency.