Closed maixhashi closed 1 month ago
このプルリクエストにて何をしたのか？ タイムラインの表示機能の切り替えをRailsのapiで再実装
このプルリクエストにて何をしたのか？
GitHub Issue で説明できない捕捉的な事項 (GitHub Issue の説明で十分であればここは不要) なぜこのプルリクエストが必要と考えたかについて説明があるとレビュワーがわかりやすい フロントエンドのjsでクラスセレクタ.activeにより表示項目を切り替えるのは挙動が安定しないため Rails apiで実装
GitHub Issue で説明できない捕捉的な事項 (GitHub Issue の説明で十分であればここは不要)
なぜこのプルリクエストが必要と考えたかについて説明があるとレビュワーがわかりやすい
.active
どの環境でどんな動作チェックをしたか 動作確認をした事についてスクショなどがあるとわかりやすくて良い 別途テスト実装予定
どの環境でどんな動作チェックをしたか
動作確認をした事についてスクショなどがあるとわかりやすくて良い
関連するプルリクエストやイシュー、コンフルリンクなど、レビュワーがレビューするにあたっての補足情報
GitHub Issue Ticket
やった事
このプルリクエストにて何をしたのか？タイムラインの表示機能の切り替えをRailsのapiで再実装なぜやるのか
GitHub Issue で説明できない捕捉的な事項 (GitHub Issue の説明で十分であればここは不要)なぜこのプルリクエストが必要と考えたかについて説明があるとレビュワーがわかりやすいフロントエンドのjsでクラスセレクタ.activeにより表示項目を切り替えるのは挙動が安定しないため Rails apiで実装動作確認
どの環境でどんな動作チェックをしたか動作確認をした事についてスクショなどがあるとわかりやすくて良い別途テスト実装予定Refs (レビューにあたって参考にすべき情報）(Optional)
関連するプルリクエストやイシュー、コンフルリンクなど、レビュワーがレビューするにあたっての補足情報