bootcamp icon indicating copy to clipboard operation
bootcamp copied to clipboard

【メンター向け】メンターが未アサインの提出物にコメントした際、担当者になったことをToastでわかりやすく通知する

Open lanactindi opened this issue 2 years ago • 7 comments

Issue

  • #5355

概要

未アサインの提出物にコメントすると、「コメントを投稿しました!」のToastしか表示されないので、担当者になったことがわかりづらいです。 (もしかすると「担当になりました。」が一瞬出ているのかもしれないが、ほとんど気づけないレベル)

Toast上でもわかりやすく表示してもらえると嬉しいです。 提出物に対して、もし担当者がいない場合、「コメントを投稿しました!」を表示しなくてもいいです。 「担当になりました。」を表示した方がいいです。 そのため、ちょっと変更しました。

  • 変更前:「担当になりました。」が表示できない(0.01秒だけ表示しただけです)、「コメントを投稿しました!」(3秒)が表示される。
  • 変更後:「担当になりました。」が表示できる(3秒になりました)、「コメントを投稿しました!」が表示されない。

確認方法

  1. feature/make_toast_notify_about_undertaking_easier_to_understandをローカルに取り込む
  2. rails sで起動する
  3. 任意の管理者ユーザーでログインする
  4. http://localhost:3000/products/1006890726 にアクセスする(どんな担当者がいない提出物もできるが、例を出しただけです。)
  5. (もし、この提出物は既に担当者が存在したら、「担当から外れる」を押してください。)
  6. 提出物詳細ページで、「コメント」部分でコメントする。
  7. 「担当になりました。」が表示される。

変更前

「担当になりました。」が表示されない。 「コメントを投稿しました!」が表示される。 Screen Shot 0004-08-25 at 17 16 14

変更後

「担当になりました。」が表示される。 Screen Shot 0004-08-25 at 17 13 08

lanactindi avatar Aug 30 '22 06:08 lanactindi

@Nabegon 先ほどのお話、 https://github.com/fjordllc/bootcamp/pull/5421#issuecomment-1231200262 新たなプルリクエストを作りました! レビューお願いします!

lanactindi avatar Aug 30 '22 06:08 lanactindi

@Nabegon ありがとうございました! @komagata すみません、こちらをレビュー頂いてお願いできますでしょうか。

lanactindi avatar Aug 30 '22 07:08 lanactindi

@lanactindi マージコミットができてしまっているので修正お願いします〜。

komagata avatar Aug 30 '22 16:08 komagata

@lanactindi マージコミットができてしまっているので修正お願いします〜。

@komagata
https://github.com/fjordllc/bootcamp/pull/5458/commits/735e711aa45b8ea7e143fcff7ad2ddff2d19a0fa こちらのコミットでしょうか?

lanactindi avatar Aug 30 '22 21:08 lanactindi

@komagata コミットを削除しましたが、buildエラーに戻ってしまいました。 https://github.com/fjordllc/bootcamp/pull/5458/checks?check_run_id=8102474062 昨日も同じエラーが出てmainブランチにマージしました。

lanactindi avatar Aug 30 '22 22:08 lanactindi

@komagata 駒形さん、修正できたので再度レビュー頂いてお願いできますでしょうか。

lanactindi avatar Aug 31 '22 14:08 lanactindi

@komagata すみません、修正できたので再度レビューくれてお願いできますでしょうか。

lanactindi avatar Sep 02 '22 10:09 lanactindi