bootcamp icon indicating copy to clipboard operation
bootcamp copied to clipboard

ダッシュボードにある最新のブックマークに削除の機能を追加した

Open choco0809 opened this issue 2 years ago • 9 comments

issue

  • https://github.com/fjordllc/bootcamp/issues/5511

概要

ダッシュボードにある最新のブックマークに編集ボタンを追加し、編集ボタンがONの時のみ削除ボタンが表示されるよう変更。

変更確認方法

  1. ブランチfeature/add_current-bookmark_delete-button_to_the_dashboardをローカルに取り込む。
  2. bin/rails sでローカル環境を立ち上げる
  3. kimuraでログインする
  4. 特定の日報をブックマークする
  5. 編集ボタンにチェックする ※ 現在、削除ボタンが表示されるとレイアウトがずれてしまいますが、ここはデザインで対応して頂く予定です。

    変更前

    _development__ダッシュボード___FBC

    変更後

    _development__ダッシュボード___FBC
  6. 削除ボタンをクリックすると、該当の日報が削除される。 _development__ダッシュボード___FBC

choco0809 avatar Sep 24 '22 06:09 choco0809

@AyakaTakashima お疲れ様です〜🙏 お手隙の際に、ご確認していただければと思います🙇‍♂️

choco0809 avatar Oct 02 '22 05:10 choco0809

@choco0809 approveした後に申し訳ございません! 先ほど、自分のPRを見ていたらkomagataさんから下記のようなコメントをいただきました🥹 https://github.com/fjordllc/bootcamp/pull/5461#discussion_r985212164

レビューした時はcapybara専用(?)のメソッドがあるとは知らず、OKしてしまったのですが 可能であればchocoさんのテストもcapybaraのメソッドに書き換えた方がいいかもしれません😭! ご参考までに🙇‍♀️

AyakaTakashima avatar Oct 03 '22 13:10 AyakaTakashima

@AyakaTakashima 有益な情報ありがとうございます😭 早速、capybaraでチェックボックスに関するメソッド発見できたので、書き換え用と思います🙏

choco0809 avatar Oct 03 '22 22:10 choco0809

@machida お疲れ様です🙇‍♂️ デザインをお手隙の際によろしくお願いします🙏

choco0809 avatar Oct 04 '22 10:10 choco0809

@choco0809 デザイン了解ですー🙋‍♂️

machida avatar Oct 04 '22 10:10 machida

@choco0809 デザイン入れましたーコンフリクト解消のため、最新のmainからrebaseしたので、手元で

git pull --rebase origin feature/add_current-bookmark_delete-button_to_the_dashboard

しておいてくださいー

machida avatar Oct 13 '22 04:10 machida

@komagata レビューお願いします🙏

machida avatar Oct 13 '22 04:10 machida

@machida

git pull --rebase origin feature/add_current-bookmark_delete-button_to_the_dashboard

了解しました🙏 デザイン対応ありがとうございます🙇

choco0809 avatar Oct 13 '22 09:10 choco0809

@komagata conflict解消しましたので、ご確認おねがいします🙏

choco0809 avatar Oct 25 '22 02:10 choco0809