-
Notifications
You must be signed in to change notification settings - Fork 98
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
fix #296 商品一覧, 商品詳細のパフォーマンス改善の動作に関して #897 #898
Conversation
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.
#897 のご指摘をもとに、もう少しわかりやすくなるよう suggestion を入れてみました。ご確認お願いいたします🙇♂️
This reverts commit 9d07ed6.
Head branch was pushed to by a user without write access
ご指摘通りの修正をコミットしました。 |
将来挙動が変わった時に検知するため
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #898 +/- ##
==========================================
+ Coverage 55.20% 55.67% +0.47%
==========================================
Files 75 75
Lines 8900 8900
==========================================
+ Hits 4913 4955 +42
+ Misses 3987 3945 -42
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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.
LGTM
ありがとうございます!良い感じですので Approve しました。 |
#897 の修正コード
意図的にはこんな感じでしょうか?