-
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
Feature/account page #115
Feature/account page #115
Conversation
503cd9e
to
31424fc
Compare
import '../../dart_flutter_common.dart'; | ||
|
||
/// タイトル、詳細、画像の3つを表示するカードウィジェット | ||
class MaterialHorizontalCard extends StatelessWidget { |
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.
実機で見ました、めちゃいい感じですね!!
}); | ||
|
||
/// [AutoRoute] で指定するパス文字列。 | ||
static const path = '/host/:userId/edit'; |
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.
いいと思います!(後からクエリパラメータで create or update が分かる情報を受け取る可能性がでるかも?なしでいけるなら全然それでも)
final hostImageUrl = ref.watch(hostImageUrlProvider(userId)); | ||
final hostDisplayName = ref.watch(hostDisplayNameProvider(userId)); | ||
final loggedInUserId = ref.watch(userIdProvider); | ||
final isMatchingUserId = loggedInUserId == userId; |
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.
「自分かどうか」ってことですね、良いと思います!
Row( | ||
children: [ | ||
FaIcon( | ||
FontAwesomeIcons.google, | ||
size: 30, | ||
), | ||
SizedBox(width: 10), | ||
Text('Google'), | ||
// TODO google連携済みかどうかで出し分けられるようにする | ||
Expanded( | ||
child: Align( | ||
alignment: Alignment.centerRight, | ||
child: Text('連携済み'), | ||
), | ||
), | ||
], | ||
), |
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.
/// 指定したユーザーの [Job] を全件取得する [FutureProvider]. | ||
final userJobsFutureProvider = | ||
FutureProvider.family.autoDispose<List<ReadJob>, String>( | ||
(ref, userId) => ref.watch(jobServiceProvider).fetchUserJobs(userId: userId), | ||
); | ||
|
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.
かんぺきです!
Issue
close #n
説明
ここに対応したことの説明や意図を書いてください。コミットの粒度が適切でコミットメッセージが十分に分かりやすければ「コミットメッセージを見てください」くらいでも構いません。
PR > Files からのソースコードへのコメントも有効なので、活用してください。
途中経過を見てほしい場合は PR のタイトルの冒頭に WIP と書いてください(Work in Progress の意味)。
UI
UI の変更がある(重要な)場合は、スクリーンショットや画面収録を貼り付けてください。
必要に応じて Before, After の画像を貼り付けるのも有効です。
その他
その他に言及したいことがあれば書いてください。
チェックリスト