[IR] コミット数のグラフ化ページを作成する

[okjsc1118]

Done definition.

1人以上のレビュー後、マージしてもらう。

What I did.

27 を実装

コンフリクトはレビュー後修正

How to check.

1. サーバを起動後、'commit_counter/index'にアクセスすると以下のグラフが表示される

2. ソースコードの'app/controller/commit_counter_controller.rb'をエディタで開き、以下の項目をそれぞれ変更
   • 9行目: 対象開発者のコミット数
   • 10行目: 対象開発者を含む全開発者のコミット数
   • 16行目: 対象開発者の名前
   • 21行目: 対象開発者を含む全開発者の人数

3. 再度1.のページにアクセスすると、グラフに2.で入力したデータが追加される

[Altairzym]

アクセスのURLはなんですか？

http://localhost:3000/commit_counter/index

ですか？このURLにアクセスすると

[Altairzym]

あ、ごめん、間違いました。。。

[okjsc1118]

@Altairzym まだ実装中だから待って笑

[Altairzym]

はい、分かりました、ごめんなさい。。。

[okjsc1118]

@yaginuuu

views/layouts/application.html.erbでincludeしてるから、assets/javascripts/配下はすべてincludeされるんだー だから、これは必要ないかも！

最初はそのつもりだった。ただライブラリとかならそれでいいんだけど、bodyの特定の位置でjs呼び出すときには必要っぽい（いろいろ試した結果）。

これってなにしてるのー？？

<%= javascript_include_tag "commit_counter" %> の記述する場合は書かなきゃダメらしい。

[yaginuuu]

@ryoki1118 includeっていうより呼び出し順の問題か！ 俺、今のプルリクでJSの呼び出し順を自由に変更できるようにしてあるから、まーじできたらここ簡潔にできるかも！ 今のところはおｋ！！

[okjsc1118]

レビューお願いします

[Altairzym]

@ryoki1118 ごめん、私の方ちょっと問題があります このタスクは私はcontrollerの方でやったのは一つプロジェクト内の全ての開発者のコミット数の集計です。 でも大木君実装したのは一人の開発者に対する各履歴プロジェクトの中のコミット数です どっちが正しいですか？

[Altairzym]

@ryoki1118 もし大木君の方の理解は正しいだったら、私controllerの方のロジックは全部間違っているかもしれないです

[Altairzym]

でもこの図は両方全部使えると思う。 グルプ分けは「プロジェクト別」と「同じプロジェクトの中の開発者別」両方全部使えると思う。

[okjsc1118]

@Altairzym 実はそこが曖昧なんだよね。この機能をどの段階で使うかによる。 使う場所はポートフォリオ機能の中のつもりだったから、開発者一人が決まってる状況で実装してる。

[yaginuuu]

これは画面2っぽいよね チケットの対象はメトリクス閲覧画面だけど

[okjsc1118]

@yaginuuu そのつもりだったけど、「ポートフォリオ機能のメトリクス閲覧画面」って、開発者とプロジェクトが両方決まってるから棒グラフで表示する意義がないよね笑

• ポートフォリオ機能のメトリクス閲覧画面
• ポートフォリオ機能の画面2
• 開発者間のメトリクス比較機能の画面1

今のところ、この3画面で使う可能性があるって認識だけどあってる？

[Altairzym]

@ryoki1118 大木君の意味が理解できる、そして図の問題もないと思います。 でもさっき言った私が「US04-F01-SP01」に対しての理解は正しいかどうか知りたいです。。。

[okjsc1118]

@Altairzym 「グラフを表示する画面によって異なる」が正解かな。 ただ仕様が厳密に決まってないから、早急に話し合って決めないと。

[Altairzym]

@ryoki1118 そうですね！決めないといけない！

[okjsc1118]

@Altairzym ある一つのプロジェクトで全開発者のコミット数を取得できるんだよね？だったら「ポートフォリオ機能のメトリクス閲覧画面」の実装形式は、ある開発者が全体のコミット数の何%を占めているかってのを棒グラフで表現すればいいかな？

[yaginuuu]

まあでも、1つできれば使い回しはできそうだからまあいいんじゃない？？

[Altairzym]

@ryoki1118 そうですね、両方全部できます！ けど、その棒グラフで　一つ開発者が一つのプロジェクトに対するコミット率だけ表示する？

[okjsc1118]

@Altairzym それだけになるね。

[Altairzym]

@ryoki1118 じゃあ、問題ないです。

[okjsc1118]

@Altairzym じゃちょっと作り変えます。

[Altairzym]

@ryoki1118 おｋ、了解です。 じゃあ、そのコミット率の棒グラフを 画面で表示する？

[Altairzym]

問題ない

[okjsc1118]

一時間ほど用事があるので一旦作業止めますが、手が空いてる人レビューお願いします。コンフリクトは後で直す予定。

[okjsc1118]

パーセント表示はグラフで表現した方がいいのかな。テキスト表示はだめ？

[yaginuuu]

tmpファイルがたくさん作られてるから削除してほしいですー！ あと、自分の環境ででtmpファイル出来ないようにしたほうがよさそう！

[Altairzym]

問題ないです！！！！