bootcamp icon indicating copy to clipboard operation
bootcamp copied to clipboard

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

Open machida opened this issue 1 year ago • 5 comments

貼り付けた画像_2024_02_14_10_36

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

machida avatar Feb 14 '24 01:02 machida

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

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

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

a-terumoto-gs avatar Feb 22 '24 05:02 a-terumoto-gs

@a-terumoto-gs

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

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

komagata avatar Feb 22 '24 11:02 komagata

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

a-terumoto-gs avatar Mar 04 '24 05:03 a-terumoto-gs

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

machida avatar Mar 04 '24 10:03 machida

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

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

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

  • https://github.com/fjordllc/bootcamp/issues/7403

a-terumoto-gs avatar Apr 03 '24 07:04 a-terumoto-gs

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

github-actions[bot] avatar Jun 05 '24 01:06 github-actions[bot]

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

a-terumoto-gs avatar Jun 06 '24 04:06 a-terumoto-gs

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

machida avatar Jun 06 '24 04:06 machida

本番環境での動作確認できたのでcloseします!

a-terumoto-gs avatar Jun 06 '24 05:06 a-terumoto-gs