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

Switch to @react-native-community/async-storage. #29

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bengtan
Copy link

@bengtan bengtan commented Jan 16, 2020

Importing AsyncStorage from react-native is deprecated. This is the replacement.

Please test this yourself before merging. I didn't test it because I don't know how to test a react native component in isolation.

@arronhunt
Copy link
Owner

Hey @bengtan

I need some additional support here before I can merge this in. AsyncStorage is bundled with RN via Expo still and the community repo requires linking which we can't do in ejected apps. I'm not sure if there's a way to support both expo and ejected applications...this is a bit above my skill level.

@bengtan
Copy link
Author

bengtan commented Feb 13, 2020

Uh, sorry, I don't know anything about expo. I've heard of it, but I've not tried using it.

If this is too troublesome, feel free to postpone it indefinitely.

@jeppan875
Copy link

How is it going with this? I want to use this package in RN web project

@arronhunt
Copy link
Owner

@jeppan875 I just added this to the v1.0.0 roadmap (#49). This will be an optional feature that will require linking for some projects.

@defrex
Copy link

defrex commented Jun 22, 2021

FWIW, Expo recommends @react-native-async-storage/async-storage for a non-deprecated alternative. That library doesn't require any additional linking.

https://docs.expo.io/versions/latest/sdk/async-storage/

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

Successfully merging this pull request may close these issues.

4 participants