java-women / ChatToolForBeginners

初心者向けチャットツールコンテンツ

5 stars 3 forks source link

Closed ihcomega56 closed 8 years ago

ihcomega56 commented 8 years ago

以下の修正でバグが出ちゃってるっぽいので直します。 https://github.com/java-women/ChatToolForBeginners/pull/3

nameが空のときにConnectボタンを押すとエラーが出る＆fooさんが接続しました。のメッセージが出ない。

→空の時はだれかが接続しました。とか表示しようかな。

zoosm3 commented 8 years ago

ななしさんが接続しました、はいかがですか？

ihcomega56 commented 8 years ago

( ﾟдﾟ)ﾊｯ!それがよいですね！

cyoco commented 8 years ago

プルリク放置しすぎて（ごめんなさい）コンフリクトしてたので、対応してマージしました。

てっきり、java側に「ななし」設定するのかと思っていて、jsで設定するって発想がなかったので、新しい発見できました＾＾

でも、基本的にはクライアントから送られてきた値は信用できないので、java側で設定するのが正しいのかなと思います。ハンズオン用のアプリだし、 #18 の対応もあるので、一旦マージしときました。

cyoco commented 8 years ago

あっ、勘違いΣ(￣ロ￣lll)　

18 の対応は関係ないですね。

ihcomega56 commented 8 years ago

@cyoco ありがとうございます〜！すみません、Java側で対応したかったんですけど正しいやり方がわからなかった…(真顔) 調べて別途リファクタリングしますね(-_-;) このissueはcloseします！