Closed robot269006 closed 7 years ago
プルリクは細かくしたほうが達成感が得られやすいと思います。
たとえば、今回の場合ですと、インクリメンタルサーチとメッセージ投稿の非同期化は別々のブランチにしたほうが、すぐにLGTMがでてやる気が出るかもしれません。
私はこのように思いますが、どちらでもいいので参考までに。
コミットのサイズが小さいとレビュワーがレビューしやすいというのもありますね。
プルリク今回一緒にあげてしまいすみませんでした。 修正対応の時間がまとまって取れなかったため一緒にあげてしまいました。 次回より気をつけます。。
上のgif でundefined となっているところは、情報をうまく渡すことができていません。 undefined は javascript ではたしか初期化されていない変数や存在しないプロパティを参照した時に出るのだったと思います。(正確ではないかもしれませんが、初期化されていない変数を参照するというのは多分あっていると思います。
undefined問題も無事解決しました。レビューいただきありがとうございます。
変更点はokです 確認として インクリメンタルサーチのgif画像と 上記のgif画像だとページの更新が行われていないかどうかわからないため、もう一度更新の有無がわかるようなmessage投稿のgif画像をお願いします
https://gyazo.com/a25e6d3af9b1b10ee1eb1acbb0da6d66 更新の有無を見るためにこの部分も写して送って頂きたいです
LGTM!!!!
WHAT
Configure asynchronous messaging function and incremental user search function
WHY
For smoother message posting and incremental user search, by the use of asynchronous communication and ajax