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

Feature/account page #115

Merged
merged 12 commits into from
Aug 9, 2023
Merged

Feature/account page #115

merged 12 commits into from
Aug 9, 2023

Conversation

narunblog
Copy link
Collaborator

Issue

close #n

説明

ここに対応したことの説明や意図を書いてください。コミットの粒度が適切でコミットメッセージが十分に分かりやすければ「コミットメッセージを見てください」くらいでも構いません。

PR > Files からのソースコードへのコメントも有効なので、活用してください。

途中経過を見てほしい場合は PR のタイトルの冒頭に WIP と書いてください(Work in Progress の意味)。

UI

UI の変更がある(重要な)場合は、スクリーンショットや画面収録を貼り付けてください。

必要に応じて Before, After の画像を貼り付けるのも有効です。

その他

その他に言及したいことがあれば書いてください。

チェックリスト

  • PR の冒頭に関連する Issue 番号を記載しましたか?
  • 本 PR の変更に関して、エディタや IDE で意図しない警告は増えていませんか?(lint 警告やタイポなど)
  • Issue の完了の定義は満たせていますか?
  • 当該 Issue のスレッドで、レビュワーにレビュー依頼をしましたか?

@narunblog narunblog changed the title 【WIP】Feature/account page Feature/account page Aug 8, 2023
import '../../dart_flutter_common.dart';

/// タイトル、詳細、画像の3つを表示するカードウィジェット
class MaterialHorizontalCard extends StatelessWidget {
Copy link
Owner

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';
Copy link
Owner

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;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

「自分かどうか」ってことですね、良いと思います!

Comment on lines +216 to +232
Row(
children: [
FaIcon(
FontAwesomeIcons.google,
size: 30,
),
SizedBox(width: 10),
Text('Google'),
// TODO google連携済みかどうかで出し分けられるようにする
Expanded(
child: Align(
alignment: Alignment.centerRight,
child: Text('連携済み'),
),
),
],
),
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

この Issue:

#124

に「やりかけてます、参考にしてください」的なメモを追加お願いしても良いですか?

Comment on lines +11 to +16
/// 指定したユーザーの [Job] を全件取得する [FutureProvider].
final userJobsFutureProvider =
FutureProvider.family.autoDispose<List<ReadJob>, String>(
(ref, userId) => ref.watch(jobServiceProvider).fetchUserJobs(userId: userId),
);

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

かんぺきです!

@narunblog narunblog merged commit 26c01b7 into main Aug 9, 2023
5 checks passed
@narunblog narunblog deleted the feature/account-page branch August 9, 2023 06:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants