管理者は定期イベント参加者を削除できるように変更

[ayu-0505]

Issue

• 8074

概要

管理者は定期イベントの参加者を削除できるように変更いたしました。

変更確認方法

1. feature/mentor_can_remove_others_from_participationをローカルに取り込む
   1. git fetch origin pull/8129/head:feature/mentor_can_remove_others_from_participation
   2. git switch feature/mentor_can_remove_others_from_participation
2. foreman start -f Procfile.devでローカルサーバーを立ち上げる
3. 管理者でログインする（例： id: komagata）
4. 参加者が登録されている定期イベントの個別ページを開く
5. 参加者一覧にて、ログインしているアカウント以外のユーザーアイコン横の削除するリンクをクリックする。
6. ページ更新後参加を取り消しました。という文言が表示され、クリックしたユーザーが参加者一覧からなくなっている。

Screenshot

変更前

変更後

追記

• 10/21 上記のPRの説明欄とissue内容が「メンターは、参加者を削除できるようにしたい。」のままになっていたので 「管理者は、参加者を削除できるようにしたい。」に修正いたしました。 合わせてスクリーンショット等を最新のものに変更しました。

[ayu-0505]

@machida さん、お疲れ様です🍵

お手隙の際にこちらの内容についてデザインを入れていただいたらと思います。 （該当部分にコメントを残します。） 「メンターのみに表示する」ことはCSSで対応できるとのことでしたので、現在はメンター以外にも表示されてしまう状態です。 抜け漏れ等ありましたらまたお知らせいただけたらと思います。 よろしくお願いします🙏

[machida]

@ayu-0505 デザイン了解ですー👍

[machida]

@ayu-0505 デザイン入れましたー

---

最新の main を取り込んだので、手元で

git pull --rebase origin feature/mentor_can_remove_others_from_participation

をお願いします。

---

ちょっと危険なリンクなので、メンターではなく管理者のみが削除を行えるように仕様を変更をさせていただきました🙇‍♂️ ほぼないですが、もしメンターがイベント登録者を削除する必要が出たときは管理者に依頼するフローにしたいと思います。 コードはほぼそのまま、1箇所だけmentorをadminにしています。

---

ご確認お願いしますー🙏

[ayu-0505]

@machidaさん、デザインありがとうございました！

管理者のみに変更になったこと、了解いたしました。 動作確認後にチームメンバーへレビュー依頼を行いたいと思います〜。

[machida]

@ayu-0505 レビュー、よろしくお願いしますー

[ayu-0505]

@machidaさん、お疲れ様です🍵

昨日お伝えしたマイグレーションファイルの件について、修正後にお知らせいただけたらと思います。 その後、メンバーレビューに進みたいと思います。 よろしくお願いします🙏

[machida]

@ayu-0505 今回の変更点を見たところ（ https://github.com/fjordllc/bootcamp/pull/8129/files ）、マイグレーションファイルもスキーマファイルも変更はなかったので、このままで大丈夫でしたー なので、このままレビューに進めちゃってくださいー

あと、ついでに、最新のmainを取り込んでおきました。 また git pull --rebase origin feature/mentor_can_remove_others_from_participation をお願いしますー

[ayu-0505]

@machida さん、ありがとうございました！ 私の方のローカルブランチでdb/schema.rbの差分が発生した問題は、コード変更によるものというより 修正の過程でたまに発生する現象の一種だったようです。 （closedされていますが、マイグレーションファイルによらないdb/schema.rbの記述をなくしたい #7543の状態と同じような問題で、bin/rails db:resetで差分が発生しなくなりました。） チェックのお時間使わせてしまいお手数をおかけしました🙏

[machida]

@ayu-0505 いえいえ、全然問題ないですー 僕もこのブランチにschemeを混ぜてしまったのかと勘違いしてしまってましたー

[ayu-0505]

@ham-cap さん、お疲れ様です🍵 レビュアー追加を繰り返して、通知を複数送ってしまいすみませんでした💦

お手数おかけしますが、こちらのPRのレビューをお願いできないでしょうか。 ご都合悪い場合は遠慮なくお知らせください🙏

[ham-cap]

@ayu-0505 承知いたしました👍 数日中には確認させていただきますので少しお待ちくださいー🙏

[ayu-0505]

@ham-cap さん、お疲れ様です🍵 確認いただきありがとうございました！ 該当コメントにも書いた通り、PR途中で要件内容が「「メンター」から「管理者」」に変わっており、 その旨をissueやPR上部コメントに記載できていなかったのを追記いたしました。 お手数をおかけしてすみませんでした🙇🏻‍♀️

改めて、またお時間ある際に確認いただけるとありがたいです。 よろしくお願いいたします🙏

[ayu-0505]

@ham-cap さん、ご確認いただきありがとうございました！

@komagata さん、お疲れ様です🍵 メンバーレビューでApproveいただいたので、またお時間ある際にご確認をお願いいたします🙏