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

日報提出お祝いメッセージの表示数を拡張する #8232

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

sugiwe
Copy link
Contributor

@sugiwe sugiwe commented Dec 2, 2024

Issue

概要

現在1000回目までのキリ番で表示される日報提出お祝いメッセージを拡張し、1024, 1100, 1111, 1200, 1234, 1300, 1400, 1500回目についても表示されるようにしました。

変更確認方法

  1. {feature/show-celebrations-when-submitting-specific-numbers-of-reports-over-1000}をローカルに取り込む
    • git fetch origin pull/8219/head:feature/show-celebrations-when-submitting-specific-numbers-of-reports-over-1000
    • git switch feature/show-celebrations-when-submitting-specific-numbers-of-reports-over-1000
  2. rails db:seedを実行する
  3. foreman start -f Procfile.dev でローカルサーバーを立ち上げる
  4. ユーザー「hajime」でログインする
  5. rails cでコンソールを開き、以下のコードを順番に実行する。
# 現在のhajimeさんの日報の数を確認
Report.where(user_id: 253826460).count
# => 5(5になっていない場合は一度アプリを落として`rails db:reset`してください)

# id 253826460、つまりhajimeさんを変数userで定義
user = User.find(253826460) 

# hajimeさんの日報が1023になるように、1018回分日報を作成する
1018.times {|n| user.reports.create!(reported_on: Time.now - 4.year + n.day, title: "test#{n}", description: "test{n}", wip: false, emotion: "happy")}
  1. トップページなどにアクセスし、自分の日報が1023になっていることを確認する
  2. /reports/newにアクセスし、任意のタイトル・内容・学習日・学習時間を入力する(1024回目の日報を追加)
  3. お祝いメッセージが表示されることを確認する

Screenshot

変更後

20241202

@sugiwe
Copy link
Contributor Author

sugiwe commented Dec 2, 2024

@machida
お疲れ様です!
こちらの件、画像追加のお願いとそもそもの追加日数をどうするかについてのご相談です。

一旦僕の方で案を考えて、1024, 1100, 1111, 1200, 1234, 1300, 1400, 1500 回目について追加で表示されるようにしています。
特筆事項としては下記です。

  • 1024: 2の10乗。プログラミングをやっていればお馴染みの番号なので
  • 1111: 1が綺麗に4つ並ぶので
  • 1234: 階段上に数字が増える綺麗な並びなので

上記については、1つ手前の数字からの近さ(例えば10001024)がちょっと気にはなりましたが、現状でも200222が存在していることや、以前「お祝いは何回あっても良い」とおっしゃっていたことも加味して上記全て入れる想定で良いかなと考えています。

一旦全てダミー画像を入れている状態です、こちらそれぞれ差し替えをしていただくというお願いの仕方で問題ないでしょうか。
また、もし追加する日数についての議論が必要でしたらおっしゃってください!

どうぞよろしくお願いいたします🙏

@machida
Copy link
Member

machida commented Dec 2, 2024

@sugiwe 提案と実装ありがとうございます!
提案通りで行きたいと思います!!
進め方も提案いただいたダミーを差し替えるで行きますー

対応ありがとうございます🙏

@sugiwe sugiwe self-assigned this Dec 9, 2024
@machida
Copy link
Member

machida commented Dec 26, 2024

@sugiwe おまたせしてすいません🙇 画像の追加と更新をしました。今回、まだお祝い設定がされてない新しい数字の画像も追加したのですが、これらは別Issueでお祝い設定を行おうと思います。それに向けてちょこちょこ画像を追加していく予定です、

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.

2 participants