特定の提出回数で表示されるお祝いメッセージは下書きの日報をカウントしないように変更、n日目からn回目に文言を変更

[Ryooo-k]

Issue

• 特定の回数で表示される日報提出お祝いメッセージが、下書きの日報もカウントしている #7885

概要

100、777、800など特定の回数で表示される日報提出お祝いメッセージが、下書き状態の日報も含めた数で表示されてしまっている。「日報提出」のお祝いメッセージのため、提出済みのみの数で表示されるように変更いたします。

変更確認方法

1. bug/daily-report-congratulatory-message-count-draft-daily-reportをローカルに取り込む i. git fetch origin pull/8123/head:bug/daily-report-congratulatory-message-count-draft-daily-report ii. git checkout bug/daily-report-congratulatory-message-count-draft-daily-report
2. rails db:seedでテストデータを作成
3. foreman start -f Procfile.devでローカルサーバーを立ち上げる
4. ユーザー名komagata、パスワードtesttestでログインを行う
5. 自分の日報にアクセスする
6. 日報の数が99個（最新の日報のみが下書きの状態です）を確認後、日報作成にアクセスする
7. 提出に必要な項目として、一意なタイトル、昨日の日付の学習時間を入力して提出を行う
8. 日報提出お祝いメッセージが表示されないことを確認する
9. 再度、日報作成にアクセスする
10. 提出に必要な項目として、一意なタイトル、今日の日付の学習時間を入力して提出を行う
11. 日報提出お祝いメッセージが表示されることを確認する

Screenshot

変更前

変更後

[Ryooo-k]

@machida お疲れ様です！ こちらのissueの修正が終わりましたので、デザインの変更をお願いいたします🙏

変更内容

• 上記写真の赤枠部のお祝いメッセージ「n日目」を「n回目」に修正お願いいたします。
• お祝いメッセージのパターンは、100, 200, 222, 300, 333, 400, 500, 555, 600, 700, 777, 800, 900, 1000日目の１４パターンありますので、全て同様に修正をお願いいたします。

[machida]

@Ryooo-k デザインの変更、了解です💪 先にレビューを進めて大丈夫ですー

[Ryooo-k]

@machida よろしくお願いします！ レビューも進めておきます。

@mousu-a お疲れ様です！ こちらのレビューをお願いできませんでしょうか🙏

[mousu-a]

@Ryooo-k お疲れ様です！

土日までにはお返しできたらと思います。 お急ぎでしたら他の方にお願いいたします。 大丈夫そうでしたらレビューさせていただきます！

ところでプルリクがDraftになっているままのようですのでopenにしてあげたほうがいいかもです〜👀

[Ryooo-k]

@mousu-a ありがとうございます！ぜひレビューお願いいたします🙇

ところでプルリクがDraftになっているままのようですのでopenにしてあげたほうがいいかもです〜👀

ご指摘ありがとうございます💦openにしました！

[Ryooo-k]

@mousu-a ご確認ありがとうございます！

こちらはリンクにしてあげるとわかりやすいかもです。 文字を範囲選択しURLをペーストするとリンクを作れます。

確かにリンクの方がわかりやすいですね！ご指摘ありがとうございます✨

「提出に必要な項目」と抽象化してしまうとレビュワー側に「提出に必要な項目ってなんだっけ🤔」といった負担を強いてしまうので、適宜具体化してあげると良さそうです。

相手がわかっていること前提で書いてしまったのが良くなかったですね💦こちらも貴重なご意見ありがとうございます！

[Ryooo-k]

@komagata お疲れ様です。 メンバーからapproveいただきましたのでご確認お願いいたします🙏

[machida]

@Ryooo-k お待たせしました！！画像の更新をしましたー

[mousu-a]

@Ryooo-k すみません🙏 「変更確認方法」についてですが、取り消し線の横に更新後のものを書く形だとkomagataさんからはわかりにくくなってしまうと思うので、更新後のものだけを残す形でいいかもです👀 例：5. 自分の日報にアクセスする

「更新されたかどうか」という情報は僕にはわかりやすくありがたいのですが、新しく見る人（komagataさん）には必要ないかもです。

[mousu-a]

すみません、closeはミスです🙇‍♂️

[Ryooo-k]

@machida デザインの修正ありがとうございます！✨

@mousu-a 確かにこの履歴はあまり残す必要は無さそうですね😅更新前の情報は見やすいように、削除しました👍 色々アドバイスありがとうございます！

[mousu-a]

@Ryooo-k お疲れ様です。 何度も申し訳ないのですが、意図していなさそうな動作を見つけました。

意図していない(?)動作

提出100回目に当たる日報を「内容修正」し「内容変更」を押すと毎回モーダルが表示されてしまうようです👀 こちら把握はされていますでしょうか？

モーダルが表示されるのは最初の1回だけで良いような気がします🤔

---

一度ご確認をお願い出来ますでしょうか🙇‍♂️

[Ryooo-k]

@mousu-a なるほど、現状だと何回もお祝いメッセージが表示されちゃいますね🧐 今回のissueで対応が必要かどうかはkomagataさんに確認してみますね👀 ご指摘ありがとうございます！

[Ryooo-k]

@mousu-a 同じお祝いメッセージが複数回表示される件、確認しましたが、対応しなくてOKとのことです🙆

@komagata 改めてのご連絡となりますがメンバーからapproveいただきましたので、レビューをお願いいたします🙏

[Ryooo-k]

@komagata 11/16ミーティングのデモ時、「〇〇回目」とお祝いメッセージが表示されず「〇〇日目」と表示されておりましたが、最新のコミットをローカルに反映してなかったため、デモの際は期待通りの動作となっていませんでした。 最新コミットを反映したところ「〇〇回目」とお祝いメッセージが表示されることを確認いたしましたので、ご報告いたします。

レビューをよろしくお願いいたします🙏

[Ryooo-k]

@komagata ご確認ありがとうございます！ ステージング環境で動作確認でき次第、テストデータを削除したコミットをpushしますので、改めてご連絡いたします。