-
Notifications
You must be signed in to change notification settings - Fork 4
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
[#234] Add secure local storage #253
Conversation
8fb1e76
to
f3fa964
Compare
@nmint8m is this PR ready for review, please? 👋 |
f3fa964
to
72bf676
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm 🚀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nmint8m As we're adding a new feature to the template, could you please help adding a new update in the feature listing from the README?
...
- Networking with [dio](https://pub.dev/packages/dio) and [retrofit](https://pub.dev/packages/retrofit), JSON serializing with [json_serializable](https://pub.dev/packages/json_serializable).
- Integrated local [secure storage](https://pub.dev/packages/flutter_secure_storage).
- [Localization](https://docs.flutter.dev/accessibility-and-localization/internationalization) integrated in [3 initial languages](bricks/template/__brick__/%7B%7Bproject_name.snakeCase()%7D%7D/lib/l10n).
...
bricks/template/__brick__/{{project_name.snakeCase()}}/pubspec.yaml
Outdated
Show resolved
Hide resolved
bricks/template/__brick__/{{project_name.snakeCase()}}/lib/storage/secure_storage.dart
Outdated
Show resolved
Hide resolved
bricks/template/__brick__/{{project_name.snakeCase()}}/lib/storage/secure_storage.dart
Outdated
Show resolved
Hide resolved
bricks/template/__brick__/{{project_name.snakeCase()}}/lib/storage/secure_storage.dart
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only @luongvo 's comments. 👍
bricks/template/__brick__/{{project_name.snakeCase()}}/pubspec.yaml
Outdated
Show resolved
Hide resolved
9aa269a
to
67bd37f
Compare
What happened 👀
In this PR I added
flutter_secure_storage
library to manage the local storage. I also created an example base storage to store and get the access token.Insight 📝
I added the
flutter_secure_storage
library. I create a singletonSecureStorage
for managing local storage. Besides, I added 1 variable to get the access token, 1 function to store the access token and 1 function for clearing all stored data.Proof Of Work 📹
N/A