Closed maixhashi closed 2 months ago
このプルリクエストにて何をしたのか？ マイページヘッダーで選択した項目によりフロントの.activeでマイページの表示項目を切り替えていたのを Railsのルーティングにより切り替えるように変更
このプルリクエストにて何をしたのか？
.active
クラスセレクタ.activeによる表示切替に関するjsコードも削除
GitHub Issue で説明できない捕捉的な事項 (GitHub Issue の説明で十分であればここは不要) なぜこのプルリクエストが必要と考えたかについて説明があるとレビュワーがわかりやすい フロントのクラスセレクタ.activeにより表示項目を切り替える利点が現状それほどないため （SPA実装しているわけでもない）
GitHub Issue で説明できない捕捉的な事項 (GitHub Issue の説明で十分であればここは不要)
なぜこのプルリクエストが必要と考えたかについて説明があるとレビュワーがわかりやすい
どの環境でどんな動作チェックをしたか 動作確認をした事についてスクショなどがあるとわかりやすくて良い テストを別途作成予定
どの環境でどんな動作チェックをしたか
動作確認をした事についてスクショなどがあるとわかりやすくて良い
関連するプルリクエストやイシュー、コンフルリンクなど、レビュワーがレビューするにあたっての補足情報
GitHub Issue Ticket
やった事
このプルリクエストにて何をしたのか？マイページヘッダーで選択した項目によりフロントの.activeでマイページの表示項目を切り替えていたのを Railsのルーティングにより切り替えるように変更クラスセレクタ.activeによる表示切替に関するjsコードも削除
なぜやるのか
GitHub Issue で説明できない捕捉的な事項 (GitHub Issue の説明で十分であればここは不要)なぜこのプルリクエストが必要と考えたかについて説明があるとレビュワーがわかりやすいフロントのクラスセレクタ.activeにより表示項目を切り替える利点が現状それほどないため （SPA実装しているわけでもない）動作確認
どの環境でどんな動作チェックをしたか動作確認をした事についてスクショなどがあるとわかりやすくて良いテストを別途作成予定Refs (レビューにあたって参考にすべき情報）(Optional)
関連するプルリクエストやイシュー、コンフルリンクなど、レビュワーがレビューするにあたっての補足情報