bootcamp
bootcamp copied to clipboard
fjordllc
ユーザー一覧の絞り込みのルールを変更。
現状
こうしたい
- 現在、「現役生」は リンク先が
/users?target=student_and_traineeになっているが、/users?target=studentに変更する。 - 「現役 + 研修生」を追加する。リンク先を
/users?target=student_and_traineeにする。（要するに、現在の「現役生」を「現役 + 研修生」にし、「現役生」を新たに追加する感じ）。
関連Issue: https://github.com/fjordllc/bootcamp/issues/7351
- 「現役生」は一般現役受講生 + 研修生を表していたが、わかりにくいので、「現役生」は一般現役受講生とすることにした。
- 「一般現役受講生 + 研修生」は「現役 + 研修生」と表すことに変更する。
@komagata さん、@machida さん お疲れさまです！ 作業を進める中で次の点が気になったので確認させてください🙏
- 「現役＋研修生」への文言変更はロケールファイルで設定するという認識でよいか
- ロケールファイルで
student_and_traineeに対応する文字列を変更すると、/admin/usersにおける絞り込みタブの表示も変更されるがよいか
- ロケールファイルで
お手すきの際にご回答いただけますと幸いです！ よろしくお願いいたします。
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
@komagata さん、@machida さん お疲れ様です。 本Issueについて本番環境での確認を行ったところ、「現役 + 研修生」タブで表示されるユーザ数が「現役生」タブと「研修生」タブの合計になっていませんでした。 「現役 + 研修生」タブで表示されるユーザーは、「現役生」タブと「研修生」タブの合計でなくてもよいのでしょうか？ ローカルやステージング環境のデータでは気づきにくい点だと感じたため、念のため確認させてください🙏
現状
「現役 + 研修生」タブの人数が「現役生」タブと「研修生」タブの合計になっていません。
具体的には、「研修生」と「卒業生」ロールが付与されているユーザーが、「現役 + 研修生」タブでは表示されず、「研修生」タブでは表示されています。
これは、「現役 + 研修生」タブのscopeが『「現役生」と「研修生」以外のロールを持っていないこと』なのに対して、「研修生」タブのscopeが『「研修生」ロールを持っていること』であることによります。
意見
- 「現役 + 研修生」タブが現在学習中の受講生を表示する形になっているのは直感的
- メンターモードでなければタブごとのユーザ数は表示されないので、一般受講生は気にならなそう
上記2点から、このままでも良いよう思いました。
@unikounio @komagata
ありがとうございます！ このままで大丈夫です。 現在、卒業生でかつ研修生の場合、手動で研修生フラグを消す対応を行っています。一人、抜けがあり数字にずれがありました。
研修生で卒業までやりきるのはイレギュラーなケースなので、当分は手動の対応でいいと考えていますが、例えば研修生一覧には現役の研修生だけを表示するなどの対応が必要になったら別Issueで対応したいと思いますー
@komagata さん、 @machida さん
ご確認ありがとうございます✨このままにさせていただきます！
ご確認いただいた件を踏まえて、本番環境での確認についてもお願いさせてください🙏
メンターモードにした際、選択したタブ名の横に表示されるユーザ数をご確認いただけますと幸いです。