okaka-c
commented
4 months ago お世話になっております。 中間レビューで頂いた内容をissue化し、ストーリーポイントを設定して 1週間ごとにissueを振り分けました。
本リリース予定日は8月16日とさせていただいております。 RSpecをどこまで実装するかによります。 以下の内容で本リリース時点でのRSpecの内容が足りなければ、また再度予定日を検討します。
■RSpec（モデルテスト・システムテスト）を書く ・RSpecを用いてモデルテスト・システムテストを各1つずつコードを記載しましょう。（テストの拡充は就活と並行して進めていきましょう。）
モデルテスト・システムテスト、各1つということで、本リリース時点では 大会情報登録(関連モデル:Competitionモデル)機能のテストまで実装したいと思いますがいかがでしょか？
- モデルスペックは大会情報登録(Competitionモデル)のバリテーションのテスト
- システムスペックは大会情報登録(Competitionモデル)周辺機能のテスト
■CI（GitHubActions） ・プッシュされたタイミングでGitHubActionsを使って、RSpec, Rubocopが実行されるようにしましょう。（継続的開発への関心をアピールする観点から）
承知しました。(8/16の本リリースレビューまでに設定します)
■グラフ表示機能 ・まだ未実装なようですので、こちらの実装も本リリースに向けて行いましょう！
承知しました。現状実装中です。今週中に実装完了見込みです
■labelとフォームの連動 ■レスポンシブ対応 ■OGP（静的） ■使用していないファイルの削除 ■Rubocop ■credentialsを使う
承知しました。本リリースまでに実装します
■（可能なら）例外が発生した際の対策 ・サービスが利用されている中で例外が発生した際に、開発者へメールでエラー内容を通知するなどをしてあげるとより良いかと思いました。
実装方法調査し、対策します
Tsuchiya2
お疲れ様です。 レビュー依頼ありがとうございます！ 以下、サービスを触ってより良くするためのレビューになります！
■RSpec（モデルテスト・システムテスト）を書く ・RSpecを用いてモデルテスト・システムテストを各1つずつコードを記載しましょう。（テストの拡充は就活と並行して進めていきましょう。）
■CI（GitHubActions） ・プッシュされたタイミングでGitHubActionsを使って、RSpec, Rubocopが実行されるようにしましょう。（継続的開発への関心をアピールする観点から）
■グラフ表示機能 ・まだ未実装なようですので、こちらの実装も本リリースに向けて行いましょう！
■labelとフォームの連動 ・labelをクリックしたら、選択された状態になるようにしましょう！
■レスポンシブ対応 ・全ページ、PC・タブレット等で閲覧されても画面に違和感が無いようにレスポンシブ対応しましょう！
■OGP（静的） ・OGP対応しましょう！
■使用していないファイルの削除 ・使用していないxxx_helper.rbなどは削除しましょう。
■Rubocop ・Rubocopを通してコードを良い状態に保ちましょう。（competition_record.rbなど）
■credentialsを使う ・config/settings/production.ymlなどに定義されているURLを、credentialsを使ってパッと見ではわからないようにしましょう。
参考：https://qiita.com/NaokiIshimura/items/2a179f2ab910992c4d39
■（可能なら）例外が発生した際の対策 ・サービスが利用されている中で例外が発生した際に、開発者へメールでエラー内容を通知するなどをしてあげるとより良いかと思いました。