masyuko0222

Results 52 comments of masyuko0222

@rira100000000 > komagataでログインしている状態だとエラーが出てアクセスできませんが、問題ないのでしょうか? 修正しました!

@rira100000000 修正いたしましたので改めてよろしくお願いいたします。

@komagata メンバーからApprove頂いたので、レビューの程よろしくお願いいたします。

@komagata お疲れ様です。ご指摘点修正しましたのでレビューのほどよろしくお願いいたします。 実装は[この会話](https://github.com/fjordllc/bootcamp/pull/7640#discussion_r1597443561)を基に行っています。 - RequestRetirement - uuid(PK)(unique) - user_id(FK) - target_user_id(FK) - String: company_name - Text: reason - Boolean: keep_data ``` User 1 -- * RequestRetirement TargetUser 1 -- 1...

> PKはなぜuuidなんでしょうか? 申請者以外がshowページにアクセスできてしまうのを避けるためです。 普通の:idを使う場合、今回は未ログインユーザーも退会申請ができるため、current_userと申請者が一致するかの確認ができず他ユーザーもshow画面にアクセスできてしまいます。 そのため:idの予測がつかないよう、uuidを利用しています。

@komagata > user_idはどうなりますか? 入力されたユーザー名やメールアドレスを基にuser_idを紐づけます。 入力されたユーザー名・メアドが本当に存在するか、またそれぞれの情報が同一のユーザーかのvalidationを行い、その後それらを基にafter_validationでusersテーブルと紐づけています。

@komagata cc: @machida ありがとうございます〜 > 一般の人の退会と同じフローで、アドバイザーの時は別のフォーム(URL)になるという感じでおねがいします。 https://github.com/fjordllc/bootcamp/pull/7640#issuecomment-2129895694 すみません、改めてIssueの認識合わせておきたいのですが、 今回は「アドバイザーが企業の他メンバーの退会申請をする」ためのフォームなので、普通の退会フォームとは異なります。 退職してしまったアドバイザーの代わりに、退会申請をしてあげる感じの機能です。 この形で実装してしまってよろしかったでしょうか? アドバイザー用の(自分の)退会フォームはまた別途Issueを建てるとのことです。 https://discord.com/channels/715806612824260640/1225666127368163380/1243598053634736249

@komagata お疲れ様です。実装が完了しましたので、レビューのほどよろしくお願いいたします。

@taco-nantai 承知しました〜! ただ僕がフロントのIssueをまだやってないので、ちょっと時間かかってしまうかもしれません😅‪‪(まだ予想も付きませんが)

@taco-nantai お疲れ様です!五月雨ですみません! 今の所、遅くて来週中のレビュー対応になるかと思います。 上記で問題なければこのままレビュー対応をさせて頂きます🙇‍♂️ 諸々のお気遣いは不要ですので、スケジュール感がご都合と合うかご確認ください〜!