-
Notifications
You must be signed in to change notification settings - Fork 1
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
Conversation
入力文章に改行が含まれるとき,改行文字が入力できないことが原因だったので,改行文字はスペースに置換するようにした.
半角ダッシュが原因.ハイフンに置換したら入力できた.
onKeyDownをウィンドウに登録
キー入力判定確認用のconsole.logを削除した.
useEffectの依存配列に足りないものを追加,関数定義をuseEffect内に移動
スコアを登録するところは素のfetchではなく |
https://github.com/su-its/typing/blob/develop/typing-app/docs/api-usage.md これ以外に詳しいドキュメントってありますか? |
七草くんがすでにclientを使うように変更してくれていた(もうマージされているらしい)ので,マージのときにfetch部分を消していただきたいです |
ありません:bow: わかりづらいですよね、ごめんなさい |
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.
api と db を docker コンテナとして立てたうえで、 yarn dev して http://localhost:3000/game にアクセスしての動作確認はしました
なのでこの PR で何かデグレってることはたぶんなくて、対数関数グラフが適切に線形のグラフに変わったし、develop ブランチから遅れていた内容も直ってると思います
気になりポイント
そこはOKなんですが、一方でhttp://localhost:3000 からログインの手順を踏んで game に移動しようとすると、うまくいきませんでした
また、別件でコンソールに画像のような警告が出ているので、もし対応できたら対応おお願いします(この PR でなくてもよいですが)
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.
GameTyping.tsx のゲージのところに瞬間タイピング速度を復活させるかどうか決めたらあとはOKです
瞬間タイピング速度の件は結局消すことにした 詳細→ #120 (comment) |
お疲れさまでした |
チケットへのリンク
resolve [FEATURE]打ち出しが反応しないことが無いように修正 #76
やったこと
やらないこと
できるようになること(ユーザ目線)
できなくなること(ユーザ目線)
動作確認
その他