LunaInsidious / chess-alpha

アルファプロジェクト用リポジトリ
2 stars 0 forks source link

最低限のプレイヤー入力画面 #4

Closed Procrustes5 closed 3 months ago

Procrustes5 commented 3 months ago

やったこと

React久しぶり・Tailwind初めてなのでレビュー厳しめにしていただけるとすごく勉強になります!

できていないこと

動作確認

https://github.com/LunaInsidious/chess-alpha/assets/66271155/9afe9e4c-0f20-445c-89cf-3bc01c13b821

notion-workspace[bot] commented 3 months ago

プレイヤー名入力画面

Procrustes5 commented 3 months ago

@LunaInsidious 修正しました! InputのFocusがすぐ外れてしまう現象があるのですが、今日限界が来ているので後日見ます🙏

Procrustes5 commented 3 months ago

@LunaInsidious 修正しました!よろしくお願いします🙏

LunaInsidious commented 3 months ago

@LunaInsidious

修正しました!よろしくお願いします🙏

今日起きたらやります!

Procrustes5 commented 3 months ago

@LunaInsidious もう一度修正しました。 domainにはplayerではなくuserで処理されているところが多かったため、playerは大体userに置き換えています。

+ CIでtestが落ちるのはなんででしょう。tsc/lintは通っていますね

LunaInsidious commented 3 months ago

@LunaInsidious

もう一度修正しました。

domainにはplayerではなくuserで処理されているところが多かったため、playerは大体userに置き換えています。

+ CIでtestが落ちるのはなんででしょう。tsc/lintは通っていますね

ありがとうございます!! image

Userの初期値でidがランダムに入るようになったのでUserに関するテストでこけてますね。

くらいが対策として思いつきますかね。

Procrustes5 commented 3 months ago

なるほど、そういうことなんですね!今回に限っては一旦テストパスするようにします🙇‍♂️