-
Notifications
You must be signed in to change notification settings - Fork 0
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
[Flutter] API Keyなどの秘匿情報の管理 #32
Conversation
…ができない問題を対応ページングができない問題を対応
!.vscode/settings.json | ||
!.vscode/launch.json | ||
!.vscode/extensions.json |
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.
チーム内で開発環境を統一するために必要なjsonファイルのみgit管理対象にする
"robert-brunhage.flutter-riverpod-snippets", // riverpod | ||
// "alexisvt.flutter-snippets", // flutter widget snippets | ||
] | ||
} |
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.
https://de-milestones.com/vscode-how-to-setup-recommended-tools/
拡張機能を統一させるための設定ファイル
] | ||
} | ||
] | ||
} |
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.
vscode上で、「実行とデバッグ」をクリックして実行する場合に指定できるビルドタイプの設定ファイル
"editor.wordBasedSuggestions": false, | ||
}, | ||
"dart.openDevTools": "flutter", | ||
} |
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.
Editorの設定やflutter fvmの設定周りの共有用の設定ファイル
"flavor": "development", | ||
"flutterAppName": "(Dev)GithubSearch", | ||
"flutterApplicationIdSuffix": ".development", | ||
"githubAccessToken": "", |
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.
秘匿情報はここの設定値をローカル環境にて各自書き換える。
@@ -14,7 +14,7 @@ final Provider githubApiProvider = Provider<GithubApi>((_) { | |||
}); | |||
|
|||
class GithubApi { | |||
static const _authority = 'api.github.com'; | |||
static const _authority = String.fromEnvironment('githubApiDomain'); |
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.
@@ -10,10 +10,11 @@ import 'package:flutter_github_search/data/api/http_handler.dart'; | |||
import 'package:flutter_github_search/domain/exception/api_exceptions.dart'; | |||
|
|||
class GithubApiHttpClient extends http.BaseClient { | |||
// static const String _accessToken = "INPUT HERE TOKEN"; | |||
static const String _accessToken = | |||
String.fromEnvironment('githubAccessToken'); |
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.
)), | ||
); | ||
} | ||
} |
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.
環境変数確認用画面
), | ||
); | ||
} | ||
} |
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.
package情報確認用画面
application idとかアプリ名とかを環境毎に変えたので。
@@ -213,6 +239,7 @@ class _SearchPagingScreenState extends ConsumerState<SearchPagingScreen> { | |||
} | |||
} | |||
} | |||
// Androidで動作 - END | |||
return false; | |||
}, | |||
); |
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.
scroll周りのバグ修正
issues (任意)
close #8
修正内容 (必須)
refs (任意)
https://zenn.dev/altiveinc/articles/separating-environments-in-flutter
https://pub.dev/packages/package_info_plus/example