masyuko0222

Results 52 comments of masyuko0222

@komagata 以下のコミットでご指摘修正いたしました。改めてよろしくお願いいたします。 811753c6424991ec0ea4cd390f646ab25d2282b6 318d82e2019811dfe2c31fd1528877c9f065f2f8

@komagata 間空いてしまってすみません。 [こちらのご指摘](https://github.com/fjordllc/bootcamp/pull/7272#discussion_r1568028016)について修正いたしました。 クラス図は以下のような形です。 ![image](https://github.com/fjordllc/bootcamp/assets/111285341/24222ce7-1917-42b5-a9eb-fcee22df36b6) - 今回は特に開催予定日に関する処理を`EventSchedule`という形で切り出しました。 - `schedule = EventSchedule.load(event_object)`とした場合、`schedule.tentative_next_event_date`や`shedule.held_next_event_date`のように利用できます。 - 内部的には`EventSchedule::SpecialEventSchedule`か`EventSchedule::RegularEventSchedule`をinitializeして、それぞれのインスタンスメソッドを呼んでいます。 作成したファイルは以下です。 - `app/models/event_schedule.rb` - `app/models/event_schedule/special_event_schedule.rb` - `app/models/event_schedule/regular_event_schedule.rb` - `app/models/event_schedule/date_calculator.rb` お手数ですが再レビューのほどよろしくお願いいたします。

@machida 仮でviewを作成しましたのでデザインのほどよろしくお願いいたします!

@machida `app/views/request_retirement/_form.html.slim` で退会申請フォームのViewを作成しましたので、デザインのほどよろしくお願いいたします。 `request_retirement/new`で閲覧できます。ログイン・未ログインどちらでもいけます。 ※`request`のほうがより適していそうだったので、`require_retirement`からルーティング変更しています。

@machida `app/views/request_retirement/show.html.slim` で退会申請完了後のshowページを作成しました。 動的に退会申請した内容を表示するようにしたので、`/new`で退会申請後に遷移するページで画面確認いただければと思いますー。

@machida たてつづきになってしまいますが、メールのViewも作成いたしましたm(__)m `app/views/user_mailer/request_retirement.html.slim`で作成していますので、デザインのほどよろしくお願いいたします。

@rira100000000 お疲れ様です。今ってレビュー受け付けていたでしょうか・・・? もしよろしければレビュー頂きたいです!

@rira100000000 うおー、すみません、帰ったらすぐ確認します! ちなみに前者のバグですが、Railsのエラー画面に遷移してしまったでしょうか? 後者についてはおっしゃる通り、管理人もアクセスできるようにしておきたいと思います。

@rira100000000 いえいえ、ご丁寧にありがとうございます! 自分もWSLなのですが細かい所でやっぱりハマりますよね汗 管理人ログインの場合については改めて調査しますので、お手数ですが引き続きレビューのほどよろしくお願いいたしますm(__)m

@rira100000000 なるほど、ありがとうございます! > そういったユーザーはそもそもアクセスする必要がないのでこれでいいのかもしれません 仰る通りレアケースであるとの、実装にかかる時間を考慮して、プルダウンで選ぶかどうかの条件分岐を`if logged_in?`ではなく`if @collegues_other_than_self && logged_in?`のようにして簡易的にエラーが出ないように修正したいと思います。