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

🚀 [@react-native-firebase/app] -Feature request: update dependencies #8213

Open
vladimirivanoviliev opened this issue Jan 6, 2025 · 5 comments
Labels
Needs Attention type: enhancement Implements a new Feature

Comments

@vladimirivanoviliev
Copy link

What feature would you like to see?

I think it would be useful to update the following dependency as it's sub dependencies are already updated to improve react-native new architecture support:

  • firebase@latest

This dependency brings update to the @firebase/auth and @firebase/auth-compat which will update @react-native-async-storage/async-storage to 2.1.0.

If I understand correctly Mike (@mikehardy, here), there is no other known issues when using the package with the react new architecture and bridge less mode.

@vladimirivanoviliev vladimirivanoviliev changed the title 🐛 @react-native-firebase/app - update dependencies 🐛 [ - update dependencies Jan 6, 2025
@vladimirivanoviliev vladimirivanoviliev changed the title 🐛 [ - update dependencies 🐛 [@react-native-firebase/app] - update dependencies Jan 6, 2025
@vladimirivanoviliev vladimirivanoviliev changed the title 🐛 [@react-native-firebase/app] - update dependencies 🚀 [@react-native-firebase/app] -Feature request: update dependencies Jan 6, 2025
@mikehardy
Copy link
Collaborator

I don't understand what the issue is. Also noting that @latest is not a valid tag for detailed discussion as it will move over time thus is ambiguous across time

@vladimirivanoviliev
Copy link
Author

vladimirivanoviliev commented Jan 6, 2025

@mikehardy we are working on moving large app to the new react-native architecture and I'm currently looking at updating related dependencies. Noticed that @react-native-async-storage/async-storage released new major version 2.1.0 (link) with improvements on the new architecture. Checked the dependency tree and it seems this dependency can be updated (in the packages that use it). Also sorry for the confusion with @latest.

@mikehardy
Copy link
Collaborator

@vladimirivanoviliev this is a devDependency (not a dependency) and should have zero impact on consumers of this library?

"devDependencies": {
"@react-native-async-storage/async-storage": "^1.24.0",
"expo": "^50.0.21"
},

Is this somehow impacting your use of react-native-async-storage/async-storage in your app such that you are not able to use v2.1.0 of that package via some resolution issue bringing in v1 ?

Regardless, sure, hadn't seen we were that out of date on it and should be updated (as devDependency and in tests app) - I just don't think it's user-visible so not sure it should be a priority vs if it was user-visible and had an impact on module consumers

@vladimirivanoviliev
Copy link
Author

vladimirivanoviliev commented Jan 6, 2025

@mikehardy After closer look it seems that I made mistake - @react-native-async-storage/async-storage is marked as optional peer dependency in firebase->firebase/auth link, which should allow me to bump it in my project without issues?

@mikehardy
Copy link
Collaborator

Test it and see

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Attention type: enhancement Implements a new Feature
Projects
None yet
Development

No branches or pull requests

2 participants