WIP状態の日報を提出する際に入力内容にエラーがある場合のバグを解消

[unikounio]

Issue

• WIPの日報を提出する際に内容にエラーがあると「すでに提出済み」のメッセージが表示される #7537

概要

WIP状態の日報を提出する際に入力内容にエラーがある場合、DB上はWIPのままであるにも関わらず、提出済み日報の編集ページが表示されてしまうバグを解消しました。

変更確認方法

1. bug/incorrect-wip-report-submission-status-messageをローカルに取り込む
2. bin/setupを実行
3. foreman start -f Procfile.devでサーバーを立ち上げる
4. ユーザー名sotugyouでログイン
5. /reportsにアクセス
6. WIP状態の日報を選択し、内容編集ボタンをクリック
7. エラー(学習時間の不正、内容の未入力など)が発生するように編集し、提出ボタンをクリック
8. 次の点を確認する
   • 適切なエラーメッセージが表示されている
   • この日報はすでに提出済みです。というメッセージが出ない
   • 提出ボタンが表示されている（内容変更ボタンがない）

Screenshot

いずれもWIP状態の日報をエラーを含むよう編集して提出ボタンを押した際の画面。

変更前

変更後

[unikounio]

@mousu-a さん
お疲れさまです！
こちらのPRのレビューをお願いさせていただきたくてご連絡させていただきました🙏
２度目のお願いとなり申し訳ないのですが、ご対応可能なようでしたら、お手すきの際にでもご確認いただけますと幸いです。
急ぎではありませんので、何卒よろしくお願いいたします。

[mousu-a]

@unikounioさん

２度目のお願いとなり申し訳ないのですが、ご対応可能なようでしたら、お手すきの際にでもご確認いただけますと幸いです。

全然ですよ〜！レビュー依頼ありがとうございます😊
2~3日以内に返せればと思っています。
それでよろしければ引き受けさせていただきます〜🙏

[mousu-a]

@unikounioさん

レビューお待たせいたしました！（2~3日と言っていたのに早くなり申し訳ないです）
コードに関して全く問題ありませんでした！ピカピカです✨
testのallow_label_click: true勉強になりました🙏

気になるところが一点ありましたのでそこに関してのコメントをさせていただきます🙏
修正必須ではないです🙆

---

「変更確認方法」について

変更確認の際に飛んで欲しいURLは/reportsの形にしてあげると親切かな、という気がします。（自分もやっていなかったので棚上げして言っています😅すみません🙏）

レビュワーは1クリックで済みますし、万が一使用しているポート番号が違う場合でも[/reports]をクリック長押しで範囲選択するだけで済みます。

/reportsちなみにこれだと、ダブルクリックではreportsまでしか範囲選択してくれないので、結局クリック長押しで範囲選択することになりそうです。
なので↑の「万が一ポート番号が違う場合でも」と同じだけのリソースを割くことになりますので、やはり↑の書き方の方が親切な気がします。どうでしょうか👀
(自分もこれを機に気をつけます🙏)

ところで「変更確認方法」8番のチェックボックス良いですね。とてもわかりやすいです😊

---

こちら修正必須ではありませんのでこちらからはApproveとさせていただきます〜🙆

[unikounio]

@mousu-a さん
迅速かつ丁寧にご確認いただきありがとうございます🙏✨早いのはむしろありがたいです！
/reportsの件、なるほどですね！ご提案いただいた形の方が良いと思いましたので、修正させていただきました。
コメント文からお気遣いを感じて嬉しくなりました😄引き続きよろしくお願いいたします！

[unikounio]

@komagata さん
お疲れ様です！
チームメンバーよりApproveをいただきましたので、お手すきの際にレビューをお願いします🙏

[komagata]

確認させていただきました。OKです～👌