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

日報提出のお祝いメッセージを特定の回数で表示する #7416

Merged

Conversation

SuzukaHori
Copy link
Contributor

@SuzukaHori SuzukaHori commented Feb 21, 2024

Issue

概要

現在100回目にしか表示されない日報提出お祝いメッセージを、100, 200, 222, 300, 333, 400, 500, 555, 600, 700, 777, 800, 900, 1000日回目の、計14回表示されるようにしました。

変更確認方法

  1. feature/show-celebration-several-times-for-report-submittionをローカルに取り込む
  2. rails db:seedを実行する
  3. foreman start -f Procfile.devでアプリを起動する
  4. ユーザ名:nippou199・パスワード:testtestでログインする
  5. /reports/newにアクセスし、任意のタイトル・内容・学習日・学習時間を入力する(200回目の日報を追加)
  6. お祝いメッセージが表示されることを確認する
  7. rails cでコンソールを開き、以下のコードを実行する。
    user = User.find(703402694)
    21.times {|n| user.reports.create!(reported_on: Time.now - 2.year + n.day, title: "test#{n}", description: "test{n}", wip: false, emotion: "happy")}
  8. /reports/newにアクセスし、任意のタイトル・内容・学習日・学習時間を入力する(222回目の日報を追加)
  9. お祝いメッセージが表示されることを確認する

Screenshot

変更後

スクリーンショット 2024-03-05 20 39 36

@SuzukaHori
Copy link
Contributor Author

@machida
お疲れ様です!
こちらの画像の追加をお願いしたいです。

存在しないのは、200, 222, 300, 333, 400, 500, 555, 600, 700, 777, 800, 900, 1000回目の、合計13個の画像です。
動作確認のため、200回目・333回目はテスト用の画像を入れているので、こちらは差し替えをお願いします。

よろしくお願いいたします🙇‍♀️

@machida
Copy link
Member

machida commented Mar 5, 2024

@SuzukaHori お待たせしました!画像を追加しましたー

@machida machida removed their assignment Mar 5, 2024
@SuzukaHori
Copy link
Contributor Author

@machida
素敵な画像の追加、ありがとうございます!!
これで毎日楽しく日報提出できそうです😊😊

一点ご確認したいのですが、333回目の画像のみ、私が作ったテスト用の画像になっています。こちらも差し替えをお願いしてもよろしいでしょうか??🙇‍♀️

@machida
Copy link
Member

machida commented Mar 5, 2024

@SuzukaHori あー、ごめんなさ、333を忘れてました!用意しますー
先にレビューを進めて大丈夫です🙆‍♂️

@SuzukaHori
Copy link
Contributor Author

@machida
確認ありがとうございます🙏
承知しました👍先にレビューを進めます!

@SuzukaHori
Copy link
Contributor Author

SuzukaHori commented Mar 5, 2024

レビュー用のメモ:

  • ステージング環境での確認用に、テストユーザを追加しています。
    複数のファイルを変更していますが、こちらは確認が終わり次第、別のPRで削除する予定です。

  • レビューでは動作確認と、以下の3つのファイルを見ていただきたいです🙏

    • app/controllers/reports_controller.rb
    • app/views/reports/_celebrate_report_count_body.html.slim
    • app/views/reports/show.html.slimの3つになります。
  • 333日目の画像のみ仮のものになっています。後で差し替えてもらう予定です。
    その関係でPRがDraftになっていますが、内部の実装は終わっています

@SuzukaHori
Copy link
Contributor Author

SuzukaHori commented Mar 5, 2024

@kurumadaisuke
お疲れさまです!
全く急ぎではないので、お手隙の際にこちらのPRのレビューをお願いできますでしょうか🙏(File changedが多いですが、見ていただきたいのは主に3つのファイルです。その他お伝えしたいことを↑に書きました。)

お時間が取れない場合は遠慮なく仰ってください。
よろしくお願いいたします🙇‍♀️

@SuzukaHori SuzukaHori requested a review from kurumadaisuke March 5, 2024 12:00
@kurumadaisuke
Copy link
Contributor

@SuzukaHori
他のissueも担当しているのでお時間かかっても大丈夫でしょうか😰?
大丈夫そうであれば見ておきます〜!!

@SuzukaHori
Copy link
Contributor Author

@kurumadaisuke
全く急ぎではないので、ご都合が良い時に見ていただければ大丈夫です!!
お忙しいところ恐れ入りますが、よろしくお願いいたします🙇‍♀️

@machida
Copy link
Member

machida commented Mar 6, 2024

@SuzukaHori お待たせしました!!忘れていた333の画像を差し替えしました🙏

@SuzukaHori
Copy link
Contributor Author

@machida
対応ありがとうございます🙏🙏
確認しました!

@SuzukaHori SuzukaHori marked this pull request as ready for review March 6, 2024 01:38
@kurumadaisuke
Copy link
Contributor

@kurumadaisuke 全く急ぎではないので、ご都合が良い時に見ていただければ大丈夫です!! お忙しいところ恐れ入りますが、よろしくお願いいたします🙇‍♀️

@SuzukaHori
こちら承諾したのに申し訳ないんですが、ローカルPCでDB周りでエラー出て手こずっていて時間が取られてしまっており、今回の内容を確認できない状況です😰(私のPCの問題ですmm)
なのでもし、お時間に余裕があるのであればこのまま私がレビュー致しますが無駄に時間経過してしまうのが申し訳ないので他の方にレビューお願いできる状況であれば他の方でもいいでしょうか?😰

@SuzukaHori
Copy link
Contributor Author

@kurumadaisuke
ご連絡ありがとうございます🙏
全然急ぎではないのですが、Fjord Choice上で手が空いている方が複数いるので、今回は別の方にお願いしようと思います🙇‍♀️

お時間をとっていただき、ありがとうございました!!

@SuzukaHori
Copy link
Contributor Author

@a-terumoto-gs
お疲れさまです!
急ぎではないので、お手隙の際にこちらのPRのレビューをお願いできますでしょうか🙏
(変更ファイルの内訳などを#7416 (comment) に書きました)

お時間が取れない場合は遠慮なく仰ってください。
よろしくお願いいたします🙇‍♀️

@SuzukaHori SuzukaHori requested review from a-terumoto-gs and removed request for kurumadaisuke and a-terumoto-gs March 7, 2024 21:49
@a-terumoto-gs
Copy link
Contributor

@SuzukaHori
お疲れ様です!レビュー依頼ありがとうございます(*'▽')

1週間以内にはレビューコメントさせていただきます!
よろしくお願いいたしますm(__)m

@a-terumoto-gs
Copy link
Contributor

@SuzukaHori
お疲れ様です!
レビュー対応が遅れてしまっており申し訳ございませんm()m
今週中には対応たいと思っていますが、もし急ぐ場合はほかの方にお願いしてもらえればと思います。
よろしくお願いいたしますm(
)m

@SuzukaHori
Copy link
Contributor Author

@a-terumoto-gs
ご連絡ありがとうございます!
今週中に対応予定の旨、承知しました👍お忙しい所恐れ入りますが、よろしくお願いいたします🙇‍♀️

Copy link
Contributor

@a-terumoto-gs a-terumoto-gs left a comment

Choose a reason for hiding this comment

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

@SuzukaHori
お疲れ様です!
確認させていただきました。
わたしからはApproveとさせていただきます!

変更確認の仕方も丁寧に書いてあってわかりやすくて助かりました(^^)/
対応が遅くなりお待たせしてしまい、申し訳ございませんでしたm(__)m

@SuzukaHori
Copy link
Contributor Author

@a-terumoto-gs
お忙しいところ、ご確認ありがとうございました🙏🙏

@komagata
メンバーにApproveいただいたので、お手隙の際にレビューをお願いします🙇‍♀️

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.

conflictの修正をお願い致します~。

@komagata komagata self-requested a review March 26, 2024 09:19
@SuzukaHori SuzukaHori force-pushed the feature/show-celebration-several-times-for-report-submittion branch from 0e1d806 to 3d9839c Compare March 26, 2024 11:54
@SuzukaHori
Copy link
Contributor Author

SuzukaHori commented Mar 26, 2024

@komagata
解消しましたので、再度ご確認をお願いします。お手数をおかけします🙇‍♀️
3/28追記:最新のコミットでコンフリクトしたので、再度修正しました。ご確認をお願いいたします🙏

@SuzukaHori SuzukaHori force-pushed the feature/show-celebration-several-times-for-report-submittion branch from 3d9839c to f800e5d Compare March 28, 2024 06:46
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です~👌

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.

5 participants