Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ユーザー一覧の絞り込みのルールを変更。 #7614

Closed
machida opened this issue Mar 29, 2024 · 9 comments
Closed

ユーザー一覧の絞り込みのルールを変更。 #7614

machida opened this issue Mar 29, 2024 · 9 comments
Assignees

Comments

@machida
Copy link
Member

machida commented Mar 29, 2024

現状

貼り付けた画像_2024_03_29_16_22

こうしたい

image
  • 現在、「現役生」は リンク先が/users?target=student_and_trainee になっているが、/users?target=studentに変更する。
  • 「現役 + 研修生」を追加する。リンク先を/users?target=student_and_traineeにする。(要するに、現在の「現役生」を「現役 + 研修生」にし、「現役生」を新たに追加する感じ)。

関連Issue: #7351

  • 「現役生」は一般現役受講生 + 研修生を表していたが、わかりにくいので、「現役生」は一般現役受講生とすることにした。
  • 「一般現役受講生 + 研修生」は「現役 + 研修生」と表すことに変更する。
@unikounio
Copy link
Contributor

@komagata さん、@machida さん
お疲れさまです!
作業を進める中で次の点が気になったので確認させてください🙏

  • 「現役+研修生」への文言変更はロケールファイルで設定するという認識でよいか
    • ロケールファイルでstudent_and_traineeに対応する文字列を変更すると、/admin/usersにおける絞り込みタブの表示も変更されるがよいか

お手すきの際にご回答いただけますと幸いです!
よろしくお願いいたします。

@komagata
Copy link
Member

@unikounio ほかのタブと同じ方法でお願いします。

Copy link

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@github-actions github-actions bot added the stale label May 29, 2024
@unikounio unikounio removed the stale label May 29, 2024
@unikounio
Copy link
Contributor

@komagata さん、@machida さん
お疲れ様です。
本Issueについて本番環境での確認を行ったところ、「現役 + 研修生」タブで表示されるユーザ数が「現役生」タブと「研修生」タブの合計になっていませんでした。
「現役 + 研修生」タブで表示されるユーザーは、「現役生」タブと「研修生」タブの合計でなくてもよいのでしょうか?
ローカルやステージング環境のデータでは気づきにくい点だと感じたため、念のため確認させてください🙏

現状

「現役 + 研修生」タブの人数が「現役生」タブと「研修生」タブの合計になっていません。
具体的には、「研修生」と「卒業生」ロールが付与されているユーザーが、「現役 + 研修生」タブでは表示されず、「研修生」タブでは表示されています。
これは、「現役 + 研修生」タブのscopeが『「現役生」と「研修生」以外のロールを持っていないこと』なのに対して、「研修生」タブのscopeが『「研修生」ロールを持っていること』であることによります。

意見

  • 「現役 + 研修生」タブが現在学習中の受講生を表示する形になっているのは直感的
  • メンターモードでなければタブごとのユーザ数は表示されないので、一般受講生は気にならなそう

上記2点から、このままでも良いよう思いました。

@komagata
Copy link
Member

komagata commented Jun 6, 2024

@machida こちらいかがでしょう~?

@machida
Copy link
Member Author

machida commented Jun 6, 2024

@unikounio @komagata

ありがとうございます!
このままで大丈夫です。
現在、卒業生でかつ研修生の場合、手動で研修生フラグを消す対応を行っています。一人、抜けがあり数字にずれがありました。

研修生で卒業までやりきるのはイレギュラーなケースなので、当分は手動の対応でいいと考えていますが、例えば研修生一覧には現役の研修生だけを表示するなどの対応が必要になったら別Issueで対応したいと思いますー

@unikounio
Copy link
Contributor

@komagata さん、 @machida さん
ご確認ありがとうございます✨このままにさせていただきます!
ご確認いただいた件を踏まえて、本番環境での確認についてもお願いさせてください🙏
メンターモードにした際、選択したタブ名の横に表示されるユーザ数をご確認いただけますと幸いです。
スクリーンショット 2024-06-06 150600

@machida
Copy link
Member Author

machida commented Jun 6, 2024

@unikounio 本番で確認しましたー🙆

@unikounio
Copy link
Contributor

@machida さん
ご確認ありがとうございます!
本番環境での確認が完了しましたので、本IssueはCloseさせていただきます。

@komagata komagata moved this to 完成 in bootcamp Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

3 participants