rira

Results 14 comments of rira

両PRともに本番環境で確認できましたのでcloseします。

@masyuko0222 レビュー依頼ありがとうございます! ローカルで動かしてみたところ、申請するボタン押下後にページが遷移しない(メールは正常に送られているようです)ので、確認をお願いします。 あと気になったのですが、 `http://localhost:3000/request_retirement/new`ですが、`komagata`でログインしている状態だとエラーが出てアクセスできませんが、問題ないのでしょうか?👀

@masyuko0222 よろしくお願いします! 申請するボタン押下後、ずっとぐるぐるして読込が終わらないような感じですね。ログにもエラー出ていないようです。 ![image](https://github.com/fjordllc/bootcamp/assets/99132547/4461ca78-4d6d-4ffc-b991-bf9246eb0c17)

@masyuko0222 すいません!画面遷移の件、こちらの環境設定で解決しました! 私はWSL環境なのでXサーバーというものでブラウザを開けるように設定する必要があるのですが、機能がオフになっていたのが原因でした。 調べたところletter_openerがメールをプレビューする際にブラウザが必要になる、ということらしいです。 お騒がせしてすみませんでした🙏

@masyuko0222 承知しました! 管理者ログイン状態でエラーが出る件ですが、こちらは再現性がありそうですが一応画面キャプチャを貼っておきます。 ![image](https://github.com/fjordllc/bootcamp/assets/99132547/85bf0527-a398-428b-9354-72d8e5e89849) 管理者が問題ではなく、`collegues_other_than_self`でnilが返ってくるユーザーがアクセスしているからだと思います。 そういったユーザーはそもそもアクセスする必要がないのでこれでいいのかもしれませんが、丁寧にやるならアクセス権限がない旨を表示したり、別に見せてはいけないページでもないので表示自体はできたりしてもいいのかな、と思いました。

@a-terumoto-gs お疲れ様です! こちらのレビューをお願いしたいのですが、ご都合いかがでしょうか?

@a-terumoto-gs 急ぎでないので、それで大丈夫です! よろしくお願いいたします🙏

@a-terumoto-gs 承知しました!別の方にお願いしたいと思います!

@SuzukaHori お疲れ様です! こちらのレビューをお願いしたいのですが、ご都合いかがでしょうか?

@SuzukaHori レビューありがとうございます! 指摘いただいた点を修正しましたので、確認をお願いいたします。