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

日報提出お祝いメッセージの表示数を拡張したい #8192

Open
sugiwe opened this issue Nov 13, 2024 · 2 comments
Open

日報提出お祝いメッセージの表示数を拡張したい #8192

sugiwe opened this issue Nov 13, 2024 · 2 comments
Assignees

Comments

@sugiwe
Copy link
Contributor

sugiwe commented Nov 13, 2024

機能の説明

#7416 で実装されている機能を拡張し、1000より多い数字でキリの良い番号についてもお祝いを表示するようにしたい

何故この機能が必要なのか

現時点で日報を1000以上提出したユーザーはいないが、学習期間の長期間傾向に伴い1000を超えるユーザーが出てくる可能性があるため
(issue起票者の @sugiwe が現在924個目の日報を提出済みで、最終的に1100ないし1111あたりまで到達する可能性があります)

お祝いを表示する日数について

#7344
上記issueでやりとりされている原案では以下となっておりましたが、
[100, 200, 222, 300, 333, 400, 500, 555, 600, 700, 777, 800, 900, 1000, 1111, 1500, 2000]

現在の実装では以下となっており、1000より大きい分は省略されているようでした。
[100, 200, 222, 300, 333, 400, 500, 555, 600, 700, 777, 800, 900, 1000]

原案との差分である3つ分を追加するか、もっと小刻みに追加するか等についても相談できれば幸いです。

@machida
Copy link
Member

machida commented Nov 15, 2024

@sugiwe なるほど!これやりましょうー

@machida machida added the 2 label Nov 15, 2024
@machida machida added this to bootcamp Nov 15, 2024
@github-project-automation github-project-automation bot moved this to やる in bootcamp Nov 15, 2024
@sugiwe sugiwe moved this from 今のイテレーション to 作業中 in bootcamp Nov 27, 2024
@sugiwe
Copy link
Contributor Author

sugiwe commented Nov 29, 2024

11/29質問タイムで確認したことメモ:

  • 2000はさすがに多すぎるので、1500あたりを上限の数値にする
  • 一旦キリ番っぽいものを見繕って提案する

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 作業中
Development

No branches or pull requests

2 participants