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

Fix/game screen/typing #120

Merged
merged 13 commits into from
Apr 16, 2024
Merged

Fix/game screen/typing #120

merged 13 commits into from
Apr 16, 2024

Conversation

Tsuyopon-1067
Copy link
Contributor

@Tsuyopon-1067 Tsuyopon-1067 commented Apr 10, 2024

チケットへのリンク

やったこと

  • 打ち出しの反応不良を修正
  • 文中での入力できない文字の修正

やらないこと

  • 無し

できるようになること(ユーザ目線)

  • 入力バグが減る

できなくなること(ユーザ目線)

  • 無し

動作確認

  • yarn buildできることを確認
% yarn build
yarn run v1.22.19
$ next build
   ▲ Next.js 14.1.0
   - Environments: .env

   Creating an optimized production build ...
 ✓ Compiled successfully
 ✓ Linting and checking validity of types    
 ✓ Collecting page data    
 ✓ Generating static pages (7/7) 
 ✓ Collecting build traces    
 ✓ Finalizing page optimization    

Route (app)                              Size     First Load JS
┌ ○ /                                    1.57 kB         258 kB
├ ○ /_not-found                          885 B          85.2 kB
├ ○ /game                                9.56 kB         139 kB
└ ○ /ranking                             1.67 kB         136 kB
+ First Load JS shared by all            84.3 kB
  ├ chunks/69-684208d8427a30a4.js        28.9 kB
  ├ chunks/fd9d1056-780053702ed842a5.js  53.4 kB
  └ other shared chunks (total)          2.04 kB


○  (Static)  prerendered as static content

✨  Done in 13.70s.

その他

  • レビュワーへの参考情報(実装上の懸念点や注意点などあれば記載)

入力文章に改行が含まれるとき,改行文字が入力できないことが原因だったので,改行文字はスペースに置換するようにした.
半角ダッシュが原因.ハイフンに置換したら入力できた.
onKeyDownをウィンドウに登録
キー入力判定確認用のconsole.logを削除した.
useEffectの依存配列に足りないものを追加,関数定義をuseEffect内に移動
@h-takeyeah
Copy link
Collaborator

スコアを登録するところは素のfetchではなく@/libs/apiからclientをimportし、それを使って実装するようにしてください

@Tsuyopon-1067
Copy link
Contributor Author

スコアを登録するところは素のfetchではなく@/libs/apiからclientをimportし、それを使って実装するようにしてください

https://github.com/su-its/typing/blob/develop/typing-app/docs/api-usage.md これ以外に詳しいドキュメントってありますか?

@Tsuyopon-1067
Copy link
Contributor Author

七草くんがすでにclientを使うように変更してくれていた(もうマージされているらしい)ので,マージのときにfetch部分を消していただきたいです

@h-takeyeah
Copy link
Collaborator

スコアを登録するところは素のfetchではなく@/libs/apiからclientをimportし、それを使って実装するようにしてください

https://github.com/su-its/typing/blob/develop/typing-app/docs/api-usage.md これ以外に詳しいドキュメントってありますか?

ありません:bow: わかりづらいですよね、ごめんなさい

Copy link
Collaborator

@h-takeyeah h-takeyeah left a comment

Choose a reason for hiding this comment

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

api と db を docker コンテナとして立てたうえで、 yarn dev して http://localhost:3000/game にアクセスしての動作確認はしました
なのでこの PR で何かデグレってることはたぶんなくて、対数関数グラフが適切に線形のグラフに変わったし、develop ブランチから遅れていた内容も直ってると思います

気になりポイント

そこはOKなんですが、一方でhttp://localhost:3000 からログインの手順を踏んで game に移動しようとすると、うまくいきませんでした

また、別件でコンソールに画像のような警告が出ているので、もし対応できたら対応おお願いします(この PR でなくてもよいですが)
image

KinjiKawaguchi
KinjiKawaguchi previously approved these changes Apr 14, 2024
h-takeyeah
h-takeyeah previously approved these changes Apr 16, 2024
Copy link
Collaborator

@h-takeyeah h-takeyeah left a comment

Choose a reason for hiding this comment

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

GameTyping.tsx のゲージのところに瞬間タイピング速度を復活させるかどうか決めたらあとはOKです

@h-takeyeah
Copy link
Collaborator

瞬間タイピング速度の件は結局消すことにした 詳細→ #120 (comment)

@h-takeyeah h-takeyeah merged commit 87c44b1 into develop Apr 16, 2024
2 checks passed
@h-takeyeah h-takeyeah deleted the fix/game-screen/typing branch April 16, 2024 13:08
@h-takeyeah
Copy link
Collaborator

お疲れさまでした

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.

[FEATURE]打ち出しが反応しないことが無いように修正
3 participants