-
Notifications
You must be signed in to change notification settings - Fork 56
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
Any further plans for improvement? #5
Comments
Thanks for the suggestion. Wait for next iteration. |
OK thanks for the fast answer. |
Let us discuss first about the usage of RxJava in our library and whether its an overkill for this project or not. cc @ragdroid |
As https://github.com/pwittchen/prefser wraps SharedPreferences, it could be used along with our library, So I don't think we need to reinvent the wheel. @vickyturtle @ragdroid What are your thoughts? |
@vashisthg Yes I agree. |
@vashisthg I don't think implementing RxJava in here makes much sense, but may be we can try to make secure-preferences injectable in https://github.com/pwittchen/prefser in place of normal SharedPreferences. |
Maybe RxJava... https://github.com/pwittchen/prefser
The text was updated successfully, but these errors were encountered: