Closed maixhashi closed 2 months ago
このプルリクエストにて何をしたのか? プロフィール画面(users#profile)でブックマークした映画を表示されるように実装
このプルリクエストにて何をしたのか?
GitHub Issue で説明できない捕捉的な事項 (GitHub Issue の説明で十分であればここは不要) なぜこのプルリクエストが必要と考えたかについて説明があるとレビュワーがわかりやすい プロフィール機能の充実化
GitHub Issue で説明できない捕捉的な事項 (GitHub Issue の説明で十分であればここは不要)
なぜこのプルリクエストが必要と考えたかについて説明があるとレビュワーがわかりやすい
どの環境でどんな動作チェックをしたか 動作確認をした事についてスクショなどがあるとわかりやすくて良い 別途テスト作成予定
どの環境でどんな動作チェックをしたか
動作確認をした事についてスクショなどがあるとわかりやすくて良い
関連するプルリクエストやイシュー、コンフルリンクなど、レビュワーがレビューするにあたっての補足情報 下記のように別々にrenderしたパーシャルファイルに対し、 横に配置するスタイルが効かなかったため position: fixed;で各ファイル内容を配置
関連するプルリクエストやイシュー、コンフルリンクなど、レビュワーがレビューするにあたっての補足情報
position: fixed;
<div> <%= render partial: 'users/bookmark_of_shuffled_overviews/bookmarked_shuffled_overview_list_on_profile', locals: { bookmarked_shuffled_overviews: @msy_bookmarked_shuffled_overviews } %> <%= render partial: 'users/bookmark_of_movies/bookmarked_movies_list_on_profile', locals: { bookmarked_movies_by_date: @bookmarked_movies_by_date } %> </div>
GitHub Issue Ticket
やった事
このプルリクエストにて何をしたのか?
プロフィール画面(users#profile)でブックマークした映画を表示されるように実装なぜやるのか
GitHub Issue で説明できない捕捉的な事項 (GitHub Issue の説明で十分であればここは不要)
なぜこのプルリクエストが必要と考えたかについて説明があるとレビュワーがわかりやすい
プロフィール機能の充実化動作確認
どの環境でどんな動作チェックをしたか
動作確認をした事についてスクショなどがあるとわかりやすくて良い
別途テスト作成予定Refs (レビューにあたって参考にすべき情報)(Optional)
関連するプルリクエストやイシュー、コンフルリンクなど、レビュワーがレビューするにあたっての補足情報
下記のように別々にrenderしたパーシャルファイルに対し、 横に配置するスタイルが効かなかったためposition: fixed;
で各ファイル内容を配置