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

提出物一覧ページの未アサインタブに各日数経過の件数を表示する #4633

Merged
merged 5 commits into from
Apr 23, 2022

Conversation

Aseiide
Copy link
Contributor

@Aseiide Aseiide commented Apr 17, 2022

issue

概要・要件

提出物一覧ページの未アサインタブに「x日経過」のラベルに()で件数を表示する

確認方法

  1. feature/display-number-of-productsブランチにチェックアウト
  2. メンター権限で提出物一覧ページ /products/unassigned にアクセス
  3. 「x日経過」のラベルに提出物の件数が表示されていることを確認する

スクリーンショット

screencapture-localhost-3000-products-unassigned-2022-04-20-14_21_34

@Aseiide Aseiide changed the title 未アサイン 未アサインタブに件数を表示する Apr 17, 2022
@Aseiide Aseiide changed the title 未アサインタブに件数を表示する [WIP]未アサインタブに件数を表示する Apr 17, 2022
@Aseiide
Copy link
Contributor Author

Aseiide commented Apr 17, 2022

screencapture-localhost-3000-products-unassigned-2022-04-17-17_30_13

@machida
未アサインのタブにだけ、件数を表示するという要件を満たすために、件数の部分だけ新たにclassを付与する必要があり、デザインが崩れてしまいました。
こちら、修正お願いできますでしょうか??

header.card-header.a-elapsed-days(v-else)
h2 {{ product_n_days_passed.elapsed_days }}日経過
.countproductsgroupedby(v-if='selectedTab === "unassigned"')
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@machida
class名も仮でつけてます。

@Aseiide Aseiide force-pushed the feature/display-number-of-products branch from 3df1acd to c17cad8 Compare April 18, 2022 06:05
@Aseiide Aseiide changed the title [WIP]未アサインタブに件数を表示する 未アサインタブに件数を表示する Apr 19, 2022
@machida
Copy link
Member

machida commented Apr 20, 2022

image

@Aseiide
デザイン調整しましたー

@machida machida removed their assignment Apr 20, 2022
@Aseiide
Copy link
Contributor Author

Aseiide commented Apr 20, 2022

@machida ありがとうございます!助かります!

@Aseiide Aseiide force-pushed the feature/display-number-of-products branch from d58a1a4 to 8f80b7c Compare April 20, 2022 03:42
@Aseiide Aseiide force-pushed the feature/display-number-of-products branch from 8f80b7c to be8af2f Compare April 20, 2022 05:13
@Aseiide Aseiide changed the title 未アサインタブに件数を表示する 提出物一覧ページの未アサインタブに各日数経過の件数を表示する Apr 20, 2022
@Aseiide Aseiide requested a review from siroemk April 20, 2022 05:32
@Aseiide
Copy link
Contributor Author

Aseiide commented Apr 20, 2022

@siroemk
お疲れさまです、こちらのレビューお願いいたします!

Copy link
Contributor

@siroemk siroemk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Aseiide
お疲れ様です〜!お待たせいたしました!
動作も修正内容も問題なさそうです。
テストの書き方など勉強になりました💪
LGTMです〜🙆‍♀️

@Aseiide
Copy link
Contributor Author

Aseiide commented Apr 22, 2022

@siroemk レビューありがとうございます!

@komagata
チームメンバーのレビューを通過したのでレビューお願いいたします。

Copy link
Member

@komagata komagata left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

確認させて頂きました。OKです〜🙆‍♂️

@komagata komagata merged commit 609a1e0 into main Apr 23, 2022
@komagata komagata deleted the feature/display-number-of-products branch April 23, 2022 16:34
@github-actions github-actions bot mentioned this pull request Apr 23, 2022
26 tasks
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.

4 participants