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

メンターが未アサインの提出物にコメントしたら、担当者になったことを画面上に反映するようにした #4518

Merged
merged 15 commits into from Apr 29, 2022
Merged

メンターが未アサインの提出物にコメントしたら、担当者になったことを画面上に反映するようにした #4518

merged 15 commits into from Apr 29, 2022

Conversation

eatplaynap
Copy link
Contributor

@eatplaynap eatplaynap commented Mar 28, 2022 •
edited

Issue概要

担当者のいない提出物にメンターがコメントすると担当者になるという機能がありますが、画面上では担当になったことがわからなかったので、画面上でもアサインが分かるように変更しました。

変更前

Image from Gyazo
コメントを行うだけで提出物の担当者にはなっているが、画面上からは通常のコメントと違いがなく、リロードしないと「担当する」ボタンが「担当から外れる」ボタンに切り替わらない

変更後

Image from Gyazo

確認方法

  1. feature/visualize-assignment-when-mentor-commented-on-unassigned-productをローカルに取り込む
  2. bin/setup実行
  3. bin/rails sでサーバーを立ち上げる
  4. メンターロールのユーザーでログイン
  5. 提出物一覧から未アサインの提出物を1つ選択し詳細画面を開く
  6. 提出物にコメントし、「担当する」ボタンが黄色の「担当から外れる」ボタンに変わり、「担当になりました」「コメントを投稿しました！」というToastが右上に表示されることを確認

@eatplaynap eatplaynap assigned eatplaynap and unassigned eatplaynap Mar 28, 2022
@eatplaynap eatplaynap requested a review from niikz March 28, 2022 14:31
@eatplaynap eatplaynap marked this pull request as ready for review March 28, 2022 14:31
@eatplaynap
Copy link
Contributor Author

@niikz
お疲れさまです！お手すきの際にこちらのレビューをお願いいたします:pray:
現在#4517 の変更がmainに取り込まれていないため、CIのテストが落ちている状態です。
それ以外の箇所で何か変なところがないかご確認いただければと思います〜！

@eatplaynap
Copy link
Contributor Author

@niikz
お疲れさまです〜！こちらのPR、本日の開発ミーティングで確認したところ、Vuexを使って書き直す必要がありそうなので、Draftに戻そうかと思います〜！
また完成したら再レビュー依頼させてください:pray:

@eatplaynap eatplaynap marked this pull request as draft March 30, 2022 13:17
@niikz
Copy link
Contributor

niikz commented Mar 30, 2022

@eatplaynap
承知いたしました〜🙆‍♀️（まだ未着手でしたので大丈夫です！）
ご連絡ありがとうございます！🙏

@eatplaynap eatplaynap force-pushed the feature/visualize-assignment-when-mentor-commented-on-unassigned-product branch 2 times, most recently from 98a1f72 to b1eccc8 Compare April 11, 2022 07:41
@eatplaynap eatplaynap marked this pull request as ready for review April 11, 2022 08:16
@eatplaynap
Copy link
Contributor Author

@niikz
お疲れさまです！やっと完成したのでお手すきの際にレビューをお願いします〜！

niikz
niikz reviewed Apr 23, 2022
View reviewed changes
Copy link
Contributor

@niikz niikz left a comment

Choose a reason for hiding this comment

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

大変お待たせしてしまいすみません〜🙇 ペアプロも見に来てくださりありがとうございました！！
コメントしましたのでお手すきの際にご確認お願いいたします！🙏

app/controllers/api/products_controller.rb Outdated
.order_for_list
.page(params[:page])
@products = if params[:id].present?
Product.where(id: params[:id])
Copy link
Contributor

Choose a reason for hiding this comment

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

こちらの条件分岐を追加した部分がよく分からなかったので教えていただきたいです〜🙇
id を指定するケースではshowアクションが適しているのかなと思いました。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

既存のjbuilderからの情報を利用するためにindex内に条件分岐の形で書いていましたが、確かに意味的にshowに書くべき内容でした！
showアクションを導入し、関連する箇所を書き換えました〜！

Copy link
Contributor

Choose a reason for hiding this comment

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

修正後のコードもとても分かりやすくなっていて理解できました！ありがとうございます😊

@niikz
Copy link
Contributor

niikz commented Apr 25, 2022

@eatplaynap
すみません、前回コメント忘れがありました🙇
descriptionで2ヵ所気付いたことがあります！

  1. 変更後のキャプチャがVuex使用前の動画とのことでしたので、こちらも修正いただければと思います！
  2. 下記はbin/setupの記載ミスでしょうか…！
  1. bin/rails setup実行

お手すきの際にご確認お願いいたします〜🙏

@eatplaynap eatplaynap force-pushed the feature/visualize-assignment-when-mentor-commented-on-unassigned-product branch from b1eccc8 to 2c5189a Compare April 26, 2022 00:17
@eatplaynap
Copy link
Contributor Author

@niikz
レビューありがとうございます！下記対応させていただきましたので、お手すきのときに再レビューをお願いいたします:pray:

  • showアクションの導入
  • descriptionの修正

@eatplaynap eatplaynap requested a review from niikz April 26, 2022 04:27
@eatplaynap eatplaynap force-pushed the feature/visualize-assignment-when-mentor-commented-on-unassigned-product branch from dd11109 to 6b94252 Compare April 26, 2022 12:20
niikz
niikz approved these changes Apr 27, 2022
View reviewed changes
Copy link
Contributor

@niikz niikz left a comment

Choose a reason for hiding this comment

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

@eatplaynap
修正ありがとうございます〜！
descriptionも確認しました！私からはOKです〜🙆‍♀️

@eatplaynap
Copy link
Contributor Author

@niikz
レビューありがとうございました😊

@komagata
チームメンバーのOKをいただいたので、お手すきの際にレビューをお願いいたします:pray:

komagata
komagata approved these changes Apr 27, 2022
View reviewed changes
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
Copy link
Member

@eatplaynap すみません、OKしてしまった後なのですが、conflictしているので解消をお願い致します〜。

@eatplaynap eatplaynap force-pushed the feature/visualize-assignment-when-mentor-commented-on-unassigned-product branch from 6b94252 to c968cd2 Compare April 27, 2022 13:34
@eatplaynap
Copy link
Contributor Author

@komagata
コンフリクト解消しましたのでマージお願いいたします:pray:

komagata
komagata approved these changes Apr 29, 2022
View reviewed changes
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 e50bbf6 into main Apr 29, 2022
@komagata komagata deleted the feature/visualize-assignment-when-mentor-commented-on-unassigned-product branch April 29, 2022 16:20
@github-actions github-actions bot mentioned this pull request Apr 29, 2022
Merged
26 tasks
@komagata
Copy link
Member

komagata commented May 3, 2022

@eatplaynap 本番環境で動作することを確認しました。

@JunichiIto JunichiIto mentioned this pull request Aug 9, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@komagata komagata komagata approved these changes

@niikz niikz niikz approved these changes

Assignees

@eatplaynap eatplaynap

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@eatplaynap @niikz @komagata