bootcamp
bootcamp copied to clipboard
fjordllc
ダッシュボードのブックマーク一覧をreactにした
Issue
- #6749
概要
ダッシュボードにあるブックマークの一覧はviewのテンプレートで実装されていましたが、それらをReactコンポーネントに置換しました。
変更確認方法
-
feature/change-bookmark-list-in-dashboard-to-reactをローカルに取り込む - 受講生、メンター、アドバイザーのアカウントでそれぞれログインし、ダッシュボードにブックマーク一覧が表示されるかを確認する。（表示されていない場合は一度DocsやQ＆A等、任意のページをブックマークし、再度トップページにアクセスする。）
- 編集ボタンを押下し、表示された削除ボタンで任意のブックマークを削除できるかを確認する。
編集ボタン
削除ボタン
削除後
Screenshot
今回のissueは既存の表示をRailsのViewファイルからReactへ置換するもののため、変更後も見た目上の変化はほとんどありませんが、一点だけ、ブックマークを削除した際に、これまでは「ブックマークを削除しました。」の表示が画面上部にフラッシュで表示されていましたが、React化に伴いトースト表示に変更していますのでご確認ください
変更前
変更後
@ham-cap お疲れ様です！レビュー承知いたしました。 他のレビュー依頼もいただいているので、1週間くらいお時間を頂戴できるとありがたいです🙇♂️すみませんが、どうぞよろしくお願いいたします。
@junohm410 お疲れ様です！ こちらの件、大変お待たせしてしまい申し訳ありません🙏 本業が繁忙期のため対応に時間がかかってしまいました💧 ご指摘いただいた点を修正いたしましたので再度ご確認いただけますでしょうか🙇♂️ よろしくお願いいたします。
@junohm410 お疲れ様です！
また、こちらはすごく小さなことなのですが、document.getElementById('bookmarks-in-dashboard')が重複しているので、DRYにできそうかな？と思いました🙏
こちら、ご指摘いただいておりました点について重複をなくして書いてみましたのでご確認ください🙏 また、コンフリクトについても解消済みです。
よろしくお願いいたします🙇♂️
@junohm410 コメントいただいた点、おっしゃるとおりですね💦 修正しておきました！ こちらこそ丁寧にレビューしていただき大変勉強になりました！ ありがとうございました💪
@komagata お疲れ様です！ メンバーのレビューがApproveになりましたのでレビューをお願いできますでしょうか🙇♂️ よろしくお願いいたします🙏
@junohm410 お疲れ様です🙏 長らく仕事がバタバタしており全く時間が取れておりませんでした。申し訳ありません🙇♂️ ご指摘のとおり不要な変更が含まれていたため除外いたしました！ すでにApproveをいただいておりますが、念のためご確認いただければ幸いです🙏
@junohm410 早速ご確認いただきありがとうございます🙏 また、長々とお付き合いいただきありがとうございました🙇♂️
@komagata 改めまして、こちらのレビューをお願いいたします🙇♂️
@machida @komagata @junohm410
こちら町田さんからお知らせいただいた内容に従い、ブックマークの件数が0になった際、残ったdivにhiddenクラスを付与する形にいたしましたのでご確認いただけますでしょうか👀
不必要なマージンは発生せず、意図したとおりの挙動になっているかと思います🙌
町田さん教えていただきありがとうございました！
こんなやり方があったとは。勉強になりました👀
@ham-cap
お疲れ様です！確認しました。
divは残りつつ、hiddenクラスの付与のおかげで余分なマージンがなくなっていることを手元で確認できました！
私も大変勉強になりました🙏ありがとうございます。
@komagata ご確認ありがとうございます🙏 今回は新しいページや要素の追加はしておらず、divのclass等もHTMLで実装されていた時と同じ構造にしているため新たにデザインをしてもらう必要はないかなと考えていたのですがいかがでしょうか👀 デザインを入れてもらう必要のあるissueを担当したことがないため思い違いをしておりましたらすみません🙏
@komagata ご確認ありがとうございます🙏 今回は新しいページや要素の追加はしておらず、divのclass等もHTMLで実装されていた時と同じ構造にしているため新たにデザインをしてもらう必要はないかなと考えていたのですがいかがでしょうか👀 デザインを入れてもらう必要のあるissueを担当したことがないため思い違いをしておりましたらすみません🙏
@ham-cap では、OKです～！
@komagata @machida お疲れ様です！ こちらのPRにつきまして、生徒・メンター・アドバイザーそれぞれの権限にて本番環境での動作確認が必要となりますので、お手数ですがメンター及びアドバイザーの確認をお願いできますでしょうか🙇♂️ 生徒については私のアカウントで動作確認済みです🙆♂️
よろしくお願いいたします🙇♂️
