bootcamp icon indicating copy to clipboard operation
bootcamp copied to clipboard

アドバイザーでログインしたときのダッシュボードに研修生がいなくても研修生を表示するブロックが表示されてしまう。

Open machida opened this issue 2 years ago • 9 comments

アドバイザーでログインすると、ログインしたアドバイザーアカウントと同じ企業に所属する現役の研修生の一覧がダッシュボードに表示される。

貼り付けた画像_2022_09_21_14_58

ログインしたアドバイザーアカウントと同じ企業に所属する現役の研修生が0の場合も、これが表示されてるので、ログインしたアドバイザーアカウントと同じ企業に所属する現役の研修生が0の場合は非表示にしたい。

machida avatar Sep 21 '22 05:09 machida

調査ログ

senpai でログインすると研修生が表示される。

_collegue_trainee.html が実際に研修生を表示している箇所

再現方法

  • kensyu を削除する
  • kensyu の所属企業を削除する

yamatatsu10969 avatar Oct 09 '22 06:10 yamatatsu10969

seed に同僚がいないユーザ作った方が良さそうな気がした。あと、テストも書いた方がいいかも。

yamatatsu10969 avatar Oct 09 '22 15:10 yamatatsu10969

@completed_learnings.present? の条件式、不要そうなんだがどうなんだろうか。 アドバイザーは企業の先輩的な立ち位置なので、終了したラーニングはなさそうだ。 実際にアドバイザーの方のデータ見たところ、終了したラーニングがない人の方が多い。

          - if current_user.belongs_company_and_adviser? && @completed_learnings.present?
            = render 'collegue_trainees', collegue_trainees: @collegue_trainees

yamatatsu10969 avatar Oct 09 '22 15:10 yamatatsu10969

issueの問題を再現するのに苦労した。 特に user に紐づく talks にもデータがないとログイン後にエラーになることを知った。 talks が nil になるので、talks_unreplied_index_url がないというエラーになる。

  1. db/fixtures/user.yml に必要なデータを入れる
  2. db/fixtures/company.yml に必要なデータを入れる
  3. db/fixures/talk.yml に必要なデータを入れる
  4. rails db:drop
  5. rails db:setup
  6. bin/rails s で起動
  7. rails c を使って、作成された user の id を探す
  8. User.find_by(login_name: 'advisernocolleguetrainee')
  9. id をつかってログインする

yamatatsu10969 avatar Oct 09 '22 15:10 yamatatsu10969

rails db:seed でも行けるんだろうか。

yamatatsu10969 avatar Oct 09 '22 15:10 yamatatsu10969

before after
CleanShot 2022-10-12 at 09 55 13@2x CleanShot 2022-10-12 at 09 56 02@2x

yamatatsu10969 avatar Oct 12 '22 00:10 yamatatsu10969

聞きたいこと

  • 簡単な再現方法
    • 今回は seed に作成して再現してみた
    • fixtures に追加した user などは、 rails seed コマンドでDBに追加されるのか
  • collegue(正しくはcolleague) をタイポしているので修正したいのだが別チケットにするべきか
  • テストはどこまで書くか問題

yamatatsu10969 avatar Oct 12 '22 01:10 yamatatsu10969

@yamatatsu10969

https://github.com/fjordllc/bootcamp/issues/5560#issuecomment-1275479350

こちらはメンターへの質問という意味でしょうか? もしそうだとしたら、メンションがないと気づくことができないので今後はメンションをお願いできれば助かります〜。 (他の方のPRやIssueも見ていただけるとどのようなコミュニケーションをとっているのか参考になるかもです〜)

独り言であれば、

:memo: xxxxx

みたいな書き方をした方が、見た人がわかりやすいかもです。

fixtures に追加した user などは、 rails seed コマンドでDBに追加されるのか

rails db:seedで追加されます。しかし、fixturesは二箇所あります。 db/fixturesはdevelopment環境のデータで、test/fixturesはtest環境のデータです。

collegue(正しくはcolleague) をタイポしているので修正したいのだが別チケットにするべきか

見つけてくださってありがとうございます。このぐらいであれば、コミットが別であれば同一Issue内で治してしまって大丈夫です〜。

komagata avatar Oct 17 '22 08:10 komagata

@komagata こちら自分用のメモなので次回から📝をつけますね!

他にも返信ありがとうございます。 疑問が解消されました。

yamatatsu10969 avatar Oct 17 '22 12:10 yamatatsu10969

📝 アドバイザーで見る必要があるので、machida さんと komagata さんに依頼中 https://discord.com/channels/715806612824260640/809595476847493192/1038393145316085780

yamatatsu10969 avatar Nov 05 '22 10:11 yamatatsu10969

確認したいただいたのでクローズ

yamatatsu10969 avatar Nov 05 '22 10:11 yamatatsu10969