search

ChallengeClub / tetris_score_server

tetris_score_server for https://github.com/seigot/tetris
MIT License
2 stars 6 forks source link

add chat function #118

Open sota1111 opened 11 months ago

sota1111 commented 11 months ago

@EndoNrak あとでコメント追記します。

EndoNrak commented 11 months ago

chat_function/data以下のファイルをgit, githubの管理下に置くかどうかがちょい迷ってます。 基本的にデータ更新の類（データベースの中身とか）は管理下に置くべきではないはずですが、今回のケースはどっちなんだろう、、、

EndoNrak commented 11 months ago

文言をちょっと修正したりするたびにPRをだす必要があるのはやだなあと思ってますが、そんなに変更するもんでもないならこれでもいいかもです

EndoNrak commented 11 months ago

それ以外はLGTMです！！

sota1111 commented 11 months ago

文言をちょっと修正したりするたびにPRをだす必要があるのはやだなあと思ってます

了解です。PR作ったらすぐコメント追記します。

sota1111 commented 11 months ago

@EndoNrak レビューお願いします。 Github ActionのFailはよく分からなかったです。

sota1111 commented 11 months ago

@EndoNrak CORS設定を追加しました。レビューお願いします。

EndoNrak commented 11 months ago

@EndoNrak CORS設定を追加しました。レビューお願いします。

OKと思います！！ ただ、dev環境のフロントエンド+バックエンドで動作確認しないと気づいてないミスがあるかもなのでmergeはそれをやってからにしたいです！ よろしくお願いします