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

メンター or 管理者でログインしたときだけユーザー一覧の期生別ページに現役生、卒業生、退会者の人数を表示する #7489

Conversation

a-terumoto-gs
Copy link
Contributor

@a-terumoto-gs a-terumoto-gs commented Mar 5, 2024

Issue

概要

ユーザー一覧ページのうち、期生別ページの全員のページに、メンター or 管理者でログインしたときだけ、現役生、卒業生、退会者の人数を表示するようにしました

変更確認方法

  1. display-number-of-category-users-on-generations-listをローカルに取り込む
  2. foreman start -f Procfile.devでサーバーを立ち上げる。
  3. komagataでログインし、/generations?target=allにアクセスする
  4. 期生別(全員)のページに現役生・卒業生・退会者の人数が表示されていることを確認する
  5. 期生別の他のページには現役生・卒業生・退会者の人数が表示がされていないことを確認する
  6. kimuraでログインしなおして、/generations?target=allにアクセスする
  7. 期生別ページに現役生・卒業生・退会者の人数が表示がされていないことを確認する

Screenshot

変更前

image

変更後

image

@a-terumoto-gs
Copy link
Contributor Author

@machida
お疲れさまです!
機能の実装とテストの追加が完了しましたので、app/javascript/generation.vueの文字列表示を追加した部分のデザインをお願いしたいです!
今はしたのような表示になっております
image
よろしくお願いいたしますm(__)m

@machida
Copy link
Member

machida commented Mar 5, 2024

@a-terumoto-gs デザイン了解ですー

@machida
Copy link
Member

machida commented Mar 16, 2024

@a-terumoto-gs
お待たせしました!デザイン入れましたー
最新のmainからrebaseしてコンフリクトを解消しておきました。

@machida machida force-pushed the feature/display-number-of-category-users-on-generations-list branch from 7b25e77 to 56ed9d5 Compare March 16, 2024 01:48
@machida machida removed their assignment Mar 16, 2024
@a-terumoto-gs a-terumoto-gs force-pushed the feature/display-number-of-category-users-on-generations-list branch 5 times, most recently from 20a68d3 to 5ccdcca Compare March 25, 2024 02:01
@a-terumoto-gs a-terumoto-gs marked this pull request as ready for review March 25, 2024 02:14
@a-terumoto-gs a-terumoto-gs requested a review from Sasaco-JM March 25, 2024 02:18
@a-terumoto-gs
Copy link
Contributor Author

@Sasaco-JM
お疲れ様です!
お手すきの際にこちらのレビューをお願いしてもよろしいでしょうか?
お忙しい等あればご遠慮なくお伝えくださいm(__)m

@Sasaco-JM
Copy link
Contributor

@a-terumoto-gs
お疲れ様です。
大変恐縮ですが時間が取れないため別の方へお願いしていただけますでしょうか。
申し訳ありませんがよろしくお願いいたします。

@a-terumoto-gs a-terumoto-gs force-pushed the feature/display-number-of-category-users-on-generations-list branch from 5ccdcca to a210fe7 Compare March 28, 2024 01:35
@a-terumoto-gs
Copy link
Contributor Author

@Sasaco-JM
feature/display-number-of-category-users-on-generations-list
お忙しいところお手数おかけして申し訳ありません。
承知いたしました。ほかの方にお願いしようと思いますm(__)m

@a-terumoto-gs a-terumoto-gs requested review from mousu-a and removed request for Sasaco-JM March 28, 2024 01:40
@a-terumoto-gs a-terumoto-gs force-pushed the feature/display-number-of-category-users-on-generations-list branch from a210fe7 to d8e0f9d Compare March 28, 2024 01:45
@a-terumoto-gs a-terumoto-gs removed the request for review from mousu-a March 29, 2024 11:47
@a-terumoto-gs a-terumoto-gs force-pushed the feature/display-number-of-category-users-on-generations-list branch from d8e0f9d to c941029 Compare April 3, 2024 06:24
@a-terumoto-gs
Copy link
Contributor Author

修正の該当ファイルが他issueで削除されてしまったため、こちらcloseします

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants