Closed Altairzym closed 9 years ago
私の場合はこんなエラーが出ましたが。
@sjc1209 ごめん、今レビュー流れを整理した
まだWIPだよね?
私はOKです。
@ryoki1118 そうですけど、IRでもいいです。。。
@ryoki1118 知りたいのは画面はこんな状態でIRになってもいいですか?
@Altairzym 終わったからレビューが欲しいのであれば[IR] 終わってないけどレビューが欲しいのであれば[WIP] 何をして欲しいのかコメントとかしてくれるとわかりやすいかな。
@ryoki1118 @sjc1209 @yaginuuu 新しいグラフはどうですか?
@Altairzym 線の上の数字って、対象の開発者が他の開発者にコメントした数だよね?線を矢印とかにできないかな。誰が誰にコメントした数なのかがわかった方がいいと思う。
@Altairzym ひとまず、動いてはいます。
@ryoki1118 それはちょっと難しいです。。。。
@ryoki1118 確かに、方向が分からないと。。。誤解するかもしれないです。。。
@Altairzym 矢印が難しいなら、対象の開発者のアイコンだけ大きくするとかでもいいかも。
@ryoki1118 うん、そうだね、やってみます
結局、矢印を追加した。。。
@Altairzym 矢印と線の統一した方が見栄えは良さそうだね。
あとページ上部に「開発者のcomments詳細表示」ってあるけど、「comments詳細」って何なのかわからないから、簡単な説明くらいは文章で書いていいんじゃない?
@ryoki1118 そうですね!じゃあ、そこで、矢印とか全部説明します
@Altairzym あ、矢印は線の色に合わせましょう。 説明はできるだけ少なくなるようにお願いします。
@ryoki1118 了解!
@Altairzym
開発者Altairzymが同じチームの他のメンバにコメントした回数が表示されている 線の太さ、上の数字と矢印の大きさで他のメンバにコメントした回数が明らかに表示されている 画像がdrag&dropできる
Altairzymのコメント数を表示します。 線上の数字は、Altairzymの各開発者に対するコメント数です。
くらいでいいんじゃない?
動作は問題ないです。
@ryoki1118 はい、分かりました
@Altairzym 修正したらマージしていいよ
マージしますね
ToDo
comments_counter/indexにアクセスすれば、こうなる
参考
review流れ
168 をレビューした時作ったデータを利用する必要がある