Skip to content

初期フローの合体をした #144

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

Merged
merged 4 commits into from
Mar 6, 2025
Merged

初期フローの合体をした #144

merged 4 commits into from
Mar 6, 2025

Conversation

Konayashi
Copy link
Collaborator

やったこと

初期フローの合体
くまちゃんやユーザの名前等の入力をする際、改行できないようにした
テキストフィールドのカラーのコードで非推奨のコードを使用していたのを修正した

Close #82

やってないこと

空文字の例外処理
通知ダイアログの実装

スクリーンショット・動作確認

自己評価

出来を 10 点満点で自己評価:

8 点

  • 実装できたのでチェックして欲しい。
  • 心配なところがいくつかある。
  • あまり理解できていないが issue などに書いてあるとおり やった。
  • その他(下に記入)

その他

  • レビュワーへの参考情報(気になった点や注意点などあれば〜)

Copy link

Tip

デバッグビルドに成功しました

Copy link

Tip

リリースビルドに成功しました

Copy link
Collaborator

@nekomoti22 nekomoti22 left a comment

Choose a reason for hiding this comment

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

バ先からLGTM

Copy link
Collaborator

@TBSten TBSten left a comment

Choose a reason for hiding this comment

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

コメントはしたけど最低限動くので LGTM

LGTM

Comment on lines +105 to +114
keyboardOptions =
KeyboardOptions.Default.copy(
imeAction = ImeAction.Done,
),
keyboardActions =
KeyboardActions(
onDone = {
keyboardController?.hide()
},
),
Copy link
Collaborator

Choose a reason for hiding this comment

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

[nits]

singleLine = true だけでやりたいことはできてそうなのでここは不要かも

Comment on lines +133 to +142
keyboardOptions =
KeyboardOptions.Default.copy(
imeAction = ImeAction.Done,
),
keyboardActions =
KeyboardActions(
onDone = {
keyboardController?.hide()
},
),
Copy link
Collaborator

Choose a reason for hiding this comment

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

[nits]
ここも singleLine = true だけで良さそう

Comment on lines +108 to +115
imeAction = ImeAction.Done,
),
keyboardActions =
KeyboardActions(
onDone = {
keyboardController?.hide()
},
),
Copy link
Collaborator

Choose a reason for hiding this comment

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

[nits]

ここも singleLine = true だけで良さそう

@Konayashi Konayashi merged commit 06d4987 into main Mar 6, 2025
1 check passed
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.

[メインタスク] 初期フロー合体の儀
3 participants