-
Notifications
You must be signed in to change notification settings - Fork 9
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
今年よく使ったオカズのランキングを年間統計ページに追加 #805
Conversation
Codecov Report
@@ Coverage Diff @@
## master #805 +/- ##
============================================
- Coverage 37.60% 37.47% -0.14%
- Complexity 787 791 +4
============================================
Files 94 94
Lines 2593 2602 +9
============================================
Hits 975 975
- Misses 1618 1627 +9
Continue to review full report at Codecov.
|
$dateSince = Carbon::minValue(); | ||
} | ||
if ($dateUntil === null) { | ||
$dateUntil = now()->addMonth()->startOfMonth(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
機能としては実害はないはずなんだけど Carbon\Carbon::setTestNow('2022-01-30')
みたいなケースでガバそう
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
確かに…
これちょっと上の関数とかも同じ感じになってて、あまりよくないですね…
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
同じオカズでチェックインボタンほしいほしくない?
@eai04191 追加しました〜 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@eai04191 やりました |
今年よく使ったオカズを振り返る枠を年間統計ページに追加します。
2回以上使っているオカズを10位まで並べます。
年末企画として直接本番に上げたいです。(なので、master宛)
Screenshot