bootcamp
bootcamp copied to clipboard
ユーザ一覧の期生別ページのそれぞれの期に人数を表示したい。
現役生、卒業生、退会者の人数を、メンター or 管理者でログインしたときだけ表示したい。
@komagata お疲れ様です。 こちらのissueの言葉の定義について確認させて下さい。 「現役生」という言葉の定義なのですが、コードを確認していたところ2種類あるのでは?と思いいたりました。 1.個人でフィヨルドに取り組まれている方のユーザーページに表示される「現役生」という属性 2.1の意味の「現役生」と企業の「研修生」を合わせた今現在フィヨルドに取り組んでいる人という意味での「現役生」という属性
ユーザー一覧ページでの、すべてタグの現役生タグでは2の属性のユーザーが表示されるようになっていると思います。 今回のissueはおなじユーザー一覧ページの期生別タグの画面の変更であるため、その仕様に合わせて、後者の2の解釈での実装と考えたのですが、この認識で間違っていないでしょうか?
もし1の意味での現役生の実装であればその旨お伝えいただけると助かります。 よろしくお願いいたします。
@a-terumoto-gs
今回のissueはおなじユーザー一覧ページの期生別タグの画面の変更であるため、その仕様に合わせて、後者の2の解釈での実装と考えたのですが、この認識で間違っていないでしょうか?
はい。 そちらでお願いします。
@komagata @machida
おつかれさまです!
現在こちらのissueについて、ユーザー一覧ページの期生別ページの全員
のタブのみに各属性の人数表示をする仕様で実装を進めています。
特に深く考えずにこの仕様にしてしまっていたのですが、、期生別ページの他のタブにも人数表示をするべきかも?と思いまして…
テストを書いていて、この辺の仕様について詰めていなかったことに気づきましたので、コメントさせていただきました。
確認よろしくお願いいたしますm(__)m
@a-terumoto-gs 質問ありがとうございます! 今回のIssueは全員に留めておいて、別のタブは別のIssueで対応しようと思いますー
@komagata @machida おつかれさまです! こちらのissueで主に変更を加えた下の3ファイルについてなのですが、
- generations.vue
- generations.js
- generation.vue メインをリベースしてきたところ、他のissueで削除されてしまっており、mainにマージ済みの状態になっていることが発覚しました。 どう対処したらよいでしょうか? 確認よろしくお願いいたしますm(__)
該当ファイルを削除しているissue
- https://github.com/fjordllc/bootcamp/issues/7403
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。
@komagata @machida 本番環境での動作確認をお願いしたいです!
@a-terumoto-gs 本番で確認しました🙆
本番環境での動作確認できたのでcloseします!