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

ユーザ一覧の期生別ページのそれぞれの期に人数を表示したい。 #7364

Closed
machida opened this issue Feb 14, 2024 · 9 comments
Assignees

Comments

@machida
Copy link
Member

machida commented Feb 14, 2024

貼り付けた画像_2024_02_14_10_36

現役生、卒業生、退会者の人数を、メンター or 管理者でログインしたときだけ表示したい。

@a-terumoto-gs
Copy link
Contributor

@komagata
お疲れ様です。
こちらのissueの言葉の定義について確認させて下さい。
「現役生」という言葉の定義なのですが、コードを確認していたところ2種類あるのでは?と思いいたりました。
1.個人でフィヨルドに取り組まれている方のユーザーページに表示される「現役生」という属性
2.1の意味の「現役生」と企業の「研修生」を合わせた今現在フィヨルドに取り組んでいる人という意味での「現役生」という属性

ユーザー一覧ページでの、すべてタグの現役生タグでは2の属性のユーザーが表示されるようになっていると思います。
今回のissueはおなじユーザー一覧ページの期生別タグの画面の変更であるため、その仕様に合わせて、後者の2の解釈での実装と考えたのですが、この認識で間違っていないでしょうか?

もし1の意味での現役生の実装であればその旨お伝えいただけると助かります。
よろしくお願いいたします。

@komagata
Copy link
Member

komagata commented Feb 22, 2024

@a-terumoto-gs

今回のissueはおなじユーザー一覧ページの期生別タグの画面の変更であるため、その仕様に合わせて、後者の2の解釈での実装と考えたのですが、この認識で間違っていないでしょうか?

はい。
そちらでお願いします。

@a-terumoto-gs
Copy link
Contributor

@komagata @machida
おつかれさまです!
現在こちらのissueについて、ユーザー一覧ページの期生別ページの全員のタブのみに各属性の人数表示をする仕様で実装を進めています。
特に深く考えずにこの仕様にしてしまっていたのですが、、期生別ページの他のタブにも人数表示をするべきかも?と思いまして…
テストを書いていて、この辺の仕様について詰めていなかったことに気づきましたので、コメントさせていただきました。
確認よろしくお願いいたしますm(__)m

@machida
Copy link
Member Author

machida commented Mar 4, 2024

@a-terumoto-gs
質問ありがとうございます!
今回のIssueは全員に留めておいて、別のタブは別のIssueで対応しようと思いますー

@a-terumoto-gs
Copy link
Contributor

a-terumoto-gs commented Apr 3, 2024

@komagata @machida
おつかれさまです!
こちらのissueで主に変更を加えた下の3ファイルについてなのですが、

  • generations.vue
  • generations.js
  • generation.vue
    メインをリベースしてきたところ、他のissueで削除されてしまっており、mainにマージ済みの状態になっていることが発覚しました。
    どう対処したらよいでしょうか?
    確認よろしくお願いいたしますm(__)

該当ファイルを削除しているissue

Copy link

github-actions bot commented Jun 5, 2024

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

@github-actions github-actions bot added the stale label Jun 5, 2024
@a-terumoto-gs
Copy link
Contributor

@komagata @machida
本番環境での動作確認をお願いしたいです!

@machida
Copy link
Member Author

machida commented Jun 6, 2024

@a-terumoto-gs 本番で確認しました🙆

@a-terumoto-gs
Copy link
Contributor

a-terumoto-gs commented Jun 6, 2024

本番環境での動作確認できたので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