natsuto6
natsuto6
@komagata こちら対応しました。お手数ですがレビューのほどよろしくお願いします🙇♂️ https://github.com/fjordllc/bootcamp/pull/7170#issuecomment-2025118933
@komagata コンフリクトを解消しました。ご確認をお願い致します。
@komagata > こちら、この別IssueはどのIssueになりますでしょうか? pagerのcomponentに変更したことにより、事象として発生しなくなっております。目的の実装になっているのでIssueとして立てておりませんでした。 @machida komagataさんからの[指摘内容](https://github.com/fjordllc/bootcamp/pull/7170#issuecomment-2045244957)について、[こちら](https://github.com/fjordllc/bootcamp/pull/7426#issuecomment-1962818085)と同様の事象が発生しております。 本番環境においても同様の事象が発生しておりますが、mainブランチでは正常に動作しております。問題の修正は既に完了しており、machidaさんに対応いただくことはない認識でよろしかったでしょうか?必要であればご対応をお願いできればと思います。お手数ですがご確認をお願い致します🙏
@machida ご対応いただきありがとうございますが解消されていることを確認できました🙇♂️ @komagata お手数ですが再レビューをお願い致します🙇♂️
@goruchanchan お疲れ様です!こちらお手数ですがレビューいただけますでしょうか? ご確認よろしくお願いします🙏
@goruchanchan ご確認いただきありがとうございます! > ちょっとしたことですが、分報チャネルのURL削除の確認で、『ユーザ自ら退会時は rails c』、『管理ページから退会させるときはユーザ登録情報変更画面』としているのは何か理由があるのでしょうか? こちらですが、画面からの確認を含めて実際のUIをを通じた動作確認を行なっても良いかなという意図で記載してました。ただ、ご指摘の通りより確実に検証できる方に統一した方がいいかなと思ったのと、レビューも煩雑になるのでRailsコンソールからの確認に修正してます🙇♂️再度お手数ですがご確認よろしくお願いします。
@goruchanchan 早々のご確認ありがとうございます!🙇♂️ ご指摘いただいていた箇所を修正したので、お手数ですが再度ご確認をお願いします🙏
@goruchanchan `test/integration/discord/users_controller_test.rb`のテストがmainブランチでは通るが、プルリクエストの変更後に失敗するという件について、`app/models/times_channel_creator.rb`で生成している`times_url`が正常に生成されていませんでした。原因は以下です 1. `ValidTimesChannel`クラスの`id`メソッドが文字列を返していたこと(これは期待される挙動ですが、テストの文脈で正しく扱われていませんでした) 2. `DISCORD_GUILD_ID`の環境変数を正しく取得できていなかったこと なのでテスト内で`ValidTimesChannel`クラスの`id`メソッドが数値としての文字列を返すように修正し、`mock_env`を使用して環境変数`DISCORD_GUILD_ID`を設定しました。[参考PR](https://github.com/fjordllc/bootcamp/pull/6970) 再度お手数ですがご確認をよろしくお願いします🙇♂️
@goruchanchan 重ねてのレビューありがとうございます!テストについておっしゃる通りでしたので、修正いたしました。 度々恐れ入りますがご確認をお願いいたします🙇♂️
@goruchanchan レビューいただきありがとうございました!!🙇♂️ @komagata メンバーレビューが通りましたので、お手数ですがレビューのほどよろしくお願い致します。