Skip to content

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

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

Closed
vladimirivanoviliev opened this issue Jan 6, 2025 · 6 comments

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

Copy link

github-actions bot commented Feb 3, 2025

Hello 👋, to help manage issues we automatically close stale issues.

This issue has been automatically marked as stale because it has not had activity for quite some time.Has this issue been fixed, or does it still require attention?

This issue will be closed in 15 days if no further activity occurs.

Thank you for your contributions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants