bootcamp
bootcamp copied to clipboard
ダッシュボードにある最新のブックマークに削除の機能を追加した
issue
- https://github.com/fjordllc/bootcamp/issues/5511
概要
ダッシュボードにある最新のブックマークに編集ボタンを追加し、編集ボタンがONの時のみ削除ボタンが表示されるよう変更。
変更確認方法
- ブランチ
feature/add_current-bookmark_delete-button_to_the_dashboard
をローカルに取り込む。 - bin/rails sでローカル環境を立ち上げる
- kimuraでログインする
- 特定の日報をブックマークする
- 編集ボタンにチェックする
※ 現在、削除ボタンが表示されるとレイアウトがずれてしまいますが、ここはデザインで対応して頂く予定です。
変更前
変更後
- 削除ボタンをクリックすると、該当の日報が削除される。
@AyakaTakashima お疲れ様です〜🙏 お手隙の際に、ご確認していただければと思います🙇♂️
@choco0809 approveした後に申し訳ございません! 先ほど、自分のPRを見ていたらkomagataさんから下記のようなコメントをいただきました🥹 https://github.com/fjordllc/bootcamp/pull/5461#discussion_r985212164
レビューした時はcapybara専用(?)のメソッドがあるとは知らず、OKしてしまったのですが 可能であればchocoさんのテストもcapybaraのメソッドに書き換えた方がいいかもしれません😭! ご参考までに🙇♀️
@AyakaTakashima 有益な情報ありがとうございます😭 早速、capybaraでチェックボックスに関するメソッド発見できたので、書き換え用と思います🙏
@machida お疲れ様です🙇♂️ デザインをお手隙の際によろしくお願いします🙏
@choco0809 デザイン了解ですー🙋♂️
@choco0809 デザイン入れましたーコンフリクト解消のため、最新のmainからrebaseしたので、手元で
git pull --rebase origin feature/add_current-bookmark_delete-button_to_the_dashboard
しておいてくださいー
@komagata レビューお願いします🙏
@machida
git pull --rebase origin feature/add_current-bookmark_delete-button_to_the_dashboard
了解しました🙏 デザイン対応ありがとうございます🙇
@komagata conflict解消しましたので、ご確認おねがいします🙏