Closed hosakou closed 4 years ago
Merging #287 into master will decrease coverage by
0.09%
. The diff coverage is0.00%
.
@@ Coverage Diff @@
## master #287 +/- ##
============================================
- Coverage 24.81% 24.72% -0.10%
- Complexity 446 448 +2
============================================
Files 112 113 +1
Lines 1358 1363 +5
============================================
Hits 337 337
- Misses 1021 1026 +5
Impacted Files | Coverage Δ | Complexity Δ | |
---|---|---|---|
app/Eloquents/Concerns/IsNewTrait.php | 0.00% <0.00%> (ø) |
2.00 <2.00> (?) |
|
app/Eloquents/Document.php | 75.00% <ø> (ø) |
3.00 <0.00> (ø) |
|
app/Eloquents/Page.php | 100.00% <ø> (ø) |
1.00 <0.00> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update a1b431e...447e440. Read the comment docs.
@hosakou レビュー遅れて申し訳ない! インターン中なので週末にできたらレビューします!
@SofPyon
修正しました!
作成から3日以内かどうか判断する関数だけ1つにまとめました!
実装内容
close #258
配布資料とお知らせでバッジを表示するようにしました
CodeIgniter では3日間表示されていたので同じ期間で作成しました
共通の処理なので1箇所にまとめて書いたほうが良かったのかなと思っているのですがどうでしょうか
懸念点
レビュワーに見て欲しい点
参考URL