bootcamp
bootcamp copied to clipboard
fjordllc
アドバイザーでログインしたときのダッシュボードに研修生がいなくても研修生を表示するブロックが表示される問題を修正
Issue
- #5560
概要
アドバイザーでログインしたときのダッシュボードに研修生がいなくても研修生を表示するブロックが表示されていたので、研修生がいない場合は表示しないようにしました。
変更確認方法
1 の方法が簡単なのでそちらで再現した方が良さそうです！
1. 既存のアドバイザーでログインして確認
- ブランチ
fix/remove_colleague_block_trainees_if_colleque_trainees_are_not_existをローカルに取り込む - senpai もしくは advijiro でログイン
- 登録者情報変更で「所属企業」を「テスト企業」に変更（テスト企業はどのユーザも所属していないため）
- ダッシュボードに研修生のブロックがないか確認
2. 今回のPRで作成した新しいアドバイザーでログインして確認
- ブランチ
fix/remove_colleague_block_trainees_if_colleque_trainees_are_not_existをローカルに取り込む - fixtures にデータを追加したので、
bin/rails seedで DB にデータを追加する -
bin/rails sでローカル環境を立ち上げる - ユーザ（User: advisernocolleguetrainee — ID: 449244843） でログイン
- ダッシュボードに研修生のブロックがないか確認
変更前
変更後
@Nabegon ~お手数ですがこちらのPRのレビューをお願いしたいです！~ ~よろしくお願いいたします🙏~
休会されているようなので他の方にレビューしていただきます🙇
@yamatatsu10969 度々失礼いたします！ 手順について確認したいです！一番目の手順の方の「既存のアドバイザーでログインして確認」についてです。
- ブランチfix/remove_colleague_block_trainees_if_colleque_trainees_are_not_existをローカルに取り込む
- senpai もしくは advijiro でログイン
- 登録者情報変更で「所属企業」を「テスト企業」に変更（テスト企業はどのユーザも所属していないため）
- ダッシュボードに研修生のブロックがないか確認
と記載されていますが、お間違えないでしょうか？
senpai、advijiroの両者どちらでログインしても、左上に「アドバイザー」と記載があります。（下記画像参照）
これはsenpai、advijiroのお二人はアドバイザーなのでは？と思いました🧐
そして、senpaiでログインした時に下記のように表示されていました！
アドバイザーのお二人（senpai、advijiro）は、研修生ではなさそう...？🧐（アドバイザーかつ研修生という状況を作り出せるのか不明ですが💦） なので、senpaiさんの会社を変更しても意味がない気がしています！ もし私の方で手順について認識の誤りがありましたらご指摘お願いします🙇♀️
@AyakaTakashima 早速手順を見ていただきありがとうございます！ 手順の通りで間違い無いです！説明が不足していて申し訳ないです。
現状の仕様と今回の修正点を整理すると、以下のようになります。
現在の仕様
- アドバイザーのダッシュボードには同じ企業に所属する研修生を表示する「研修生ブロック」が存在する
- ブロックというのは「最新のお知らせ」「就職活動中のユーザー」のような箇所です
- 同じ企業に研修生がいる場合もいない場合も「研修生ブロック」が表示される
今回の修正点
- アドバイザーでログインしている時、同じ企業に研修生がいない場合は「研修生ブロック」は表示しない
研修生ブロックには全ての研修生が表示されるのではなく、同じ企業に所属している研修生だけが表示されることを書いておけばよかったです🙇♂️
アドバイザーのお二人（senpai、advijiro）は、研修生ではなさそう...？🧐（アドバイザーかつ研修生という状況を作り出せるのか不明ですが💦） なので会社を変更しても意味がない気がしています！
【アドバイザーでログインしている時、同じ企業に研修生がいない場合は「研修生ブロック」は表示しない】を再現するために、以下のような手順を踏んでおります！
- アドバイザーでログイン
- 同じ企業に所属する研修生がいないようにするために、所属企業を変更
また、疑問点やわかりにくい点がありましたらお声がけください。 ペアプロなども可能です。 よろしくお願い致します！
@yamatatsu10969 なるほど！下記の点を勘違いしておりました💦
研修生ブロックには全ての研修生が表示されるのではなく、同じ企業に所属している研修生だけが表示されることを書いておけばよかったです🙇♂️
それであればこの確認手順で大丈夫そうです👌無事確認できました！ ご説明ありがとうございます！🙇♀️
@AyakaTakashima あー！申し訳ないです。。。そちらも同じ文言を追加していたのを失念していました。。 修正しましたので再度ご確認お願い致します🙇♂️ https://github.com/fjordllc/bootcamp/pull/5664/commits/1be8c20112a77737e676d72a12ccda7afd98a32f