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

提出物の自分の担当タブの絞り込みの順番を逆にする #7156

Conversation

naokinaokiboo
Copy link
Contributor

Issue

概要

提出物の「自分の担当」タブの絞り込みの順番を逆(「全て」を左、「未返信」を右)にしました。

変更確認方法

  1. ブランチfeature/reverse-refinement-order-in-responsibilities-tabs-of-submissionsをローカルに取り込む
  2. サーバーを立ち上げ、メンターロールでhttp://localhost:3000/products/self_assigned にアクセスする。
  3. 絞り込み用のボタンの配置を確認する(「全て」が左側に、「未返信」が右側に配置されていること)

Screenshot

変更前

image

変更後

image

@naokinaokiboo naokinaokiboo self-assigned this Dec 20, 2023
@naokinaokiboo
Copy link
Contributor Author

@junohm410
こんにちは。こちらのレビューをお願いすることは可能でしょうか?🙏
もしもご都合悪ければ、その旨、仰っていただけたらと思います。
よろしくお願い致します。🙇‍♂️

@junohm410
Copy link
Contributor

@naokinaokiboo
お疲れ様です!お返事遅くなりすみませんでした🙏
レビューの件、承知しました。金曜日までにはお返事できるようにします🙏

@junohm410
Copy link
Contributor

@naokinaokiboo
お疲れ様です。レビューさせていただきました。
変更前・変更後を共に確認し、絞り込みボタンの配置が変更確認方法の記載のとおりになっていることを確認いたしました🤝
スクショのおかげで確認点もわかりやすくてありがたかったです🙏
私からはApproveとさせていただきます🤝

@naokinaokiboo
Copy link
Contributor Author

@junohm410
お疲れ様です。お忙しい中、早速のレビューありがとうございました😊🙏

@naokinaokiboo
Copy link
Contributor Author

@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 ac20926 into main Dec 26, 2023
5 checks passed
@komagata komagata deleted the feature/reverse-refinement-order-in-responsibilities-tabs-of-submissions branch December 26, 2023 23:16
@komagata
Copy link
Member

@naokinaokiboo 本番環境で確認してOKでした〜。

@github-actions github-actions bot mentioned this pull request Jan 13, 2024
28 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.

3 participants