Closed MikotoMakizuru closed 2 weeks ago
@ayu-0505 お疲れ様です、コードレビューをお願いすることは可能でしょうか。 よろしくお願いいたします。
@MikotoMakizuru さん、お疲れ様です🍵
了解しました! 1週間を目処(最低でも来週4日月曜)に確認いたしますので、少しお待ちください🙏 よろしくお願いいたします。
@MikotoMakizuru さん、お疲れ様です🍵
確認いたしました〜!
現在のコードで気になる部分は見当たりません。
ただ、モデルに新しいメソッドを追加しておりますので、ユニットテストも作成したほうが良いかなと考えます。
追加したい機能そのものについてはシステムテストで担保されていますが、将来他の目的でcancel_participation_from_regular_events
メソッドを使用する可能性がありますので、メソッド自体の動作の担保が必要かも、という風に思っています。
上記の件、確認のほどよろしくお願いします🙏
@ayu-0505 レビューありがとうございます。
将来他の目的でcancel_participation_from_regular_eventsメソッドを使用する可能性がありますので、メソッド自体の動作の担保が必要かも、という風に思っています。
確かにそうですね。長期的に web 開発を行っていくうえで cancel_participation_from_regular_events
メソッドのテストも必要かと私も思います。
テスト追加しましたのでご確認お願いします https://github.com/fjordllc/bootcamp/pull/8152/commits/95d039a8eae64edefa6c2dc80dab1acbf5480370
@MikotoMakizuru さん、修正ありがとうございました🙏
私のほうでは他に気になる点はありませんので、Approveとさせていただきます。
@ayu-0505 レビュー、ありがとうございます。🙇♂️
@komagata お疲れ様です、メンバーレビューが終わりましたのでレビューお願いしたいです!
:memo: @okuramasafumi さんにレビューをお願いしました。
Issue
8073
概要
休会・退会したら参加している定期イベントはキャンセルされるようにしました
変更確認方法
事前準備
feature/on-hibernation-remove-from-regular-events
をローカルに取り込むrails db:seed
を実行して初期データ投入(このコマンドはローカルのDBを初期化します。初期化しても問題ないタイミングで行ってください。)http://localhost:3000/regular_events/459650222
)に遷移休会時の確認手順
退会時の確認手順
Screenshot
変更前
休会(または退会)したユーザが定期イベントに残っている
https://github.com/user-attachments/assets/8340e8a7-7704-44b1-a01a-c6c078993617
変更後
休会したら参加している定期イベントからいなくなる
https://github.com/user-attachments/assets/73e24833-a8ea-4a90-a957-fcec3b092534
退会したら参加している定期イベントからいなくなる
https://github.com/user-attachments/assets/3868c4d7-1f62-48b9-b026-c8864fc1658f