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.

Sign up for GitHub

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

Jump to bottom

【メンター向け】「n日経過」の部分に件数を入れたい #4345

Closed
JunichiIto opened this issue Mar 3, 2022 · 9 comments
Closed

【メンター向け】「n日経過」の部分に件数を入れたい #4345

JunichiIto opened this issue Mar 3, 2022 · 9 comments
Assignees
@Aseiide
Labels
チャレンジレベル2

Comments

@JunichiIto
Copy link
Contributor

提出物の未アサインタブで「5日経過」のような区切り線が表示されるが、件数が多いと具体的な件数がが把握しづらいので、「5日経過 (10)」のように件数が表示されると嬉しい。

hoge
@machida machida mentioned this issue Mar 4, 2022
Closed
This was referenced Mar 14, 2022
Merged
Closed
@Aseiide
Copy link
Contributor

Aseiide commented Apr 6, 2022

開発MTG 📝
未アサインと未完了、どちらにも表示する

Aseiide added a commit that referenced this issue Apr 13, 2022
@Aseiide
#4345 で対応するためテストを削除
3848993
Aseiide added a commit that referenced this issue Apr 13, 2022
@Aseiide
#4345 で対応するためテストを削除
bddb051
@Aseiide Aseiide mentioned this issue Apr 14, 2022
Closed
@Aseiide
Copy link
Contributor

Aseiide commented Apr 15, 2022

@machida
こちらなんですけど、件数は「未完了」と「未アサイン」のどちらにもあったほうが良いですか? それとも、「未アサイン」タブだけに表示しますか??

@machida
Copy link
Member

machida commented Apr 15, 2022

両方にお願いしますー

@Aseiide
Copy link
Contributor

Aseiide commented Apr 15, 2022

@machida わかりました！ありがとうございます！

@Aseiide
Copy link
Contributor

Aseiide commented Apr 16, 2022 •
edited
Loading

1ページ

screencapture-localhost-3000-products-unchecked-2022-04-16-14_30_27

2ページ

screencapture-localhost-3000-products-unchecked-2022-04-16-14_30_45

@machida
「未完了」のタブにはページネーションがあって、今の実装だとページが変わると表示される件数が変わってしまうんですが、()の中には全件数が表示されたほうが良いですよね??
この場合だったら、ページ1にもページ2にも1日経過の全件数である(48)を表示すると思っているんですが。

@Aseiide
Copy link
Contributor

Aseiide commented Apr 16, 2022

📝

  • jbuilderで全件数を取って、それをvueで表示する
  • integrationテストでレスポンスを確認する
  • システムテストで書く

@machida
Copy link
Member

machida commented Apr 16, 2022

あー、なるほど、ではページネーションのない未アサインタブだけで大丈夫ですー🙆‍♂️

@Aseiide
Copy link
Contributor

Aseiide commented Apr 16, 2022

@machida
承知しました！

@Aseiide Aseiide mentioned this issue Apr 17, 2022
Merged
@Aseiide
Copy link
Contributor

Aseiide commented May 3, 2022

動作確認ができたのでcloseします！ @machida 確認ありがとうございました！

@Aseiide Aseiide closed this as completed May 3, 2022
@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
Assignees

@Aseiide Aseiide

Labels
チャレンジレベル2
Projects
bootcamp
Archived in project
Milestone
No milestone
Development

No branches or pull requests
4 participants
@komagata @machida @JunichiIto @Aseiide