-
-
Notifications
You must be signed in to change notification settings - Fork 211
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
chore: Add Cookpad Case Study #1176
chore: Add Cookpad Case Study #1176
Conversation
Please provide feedback if the summary written for this case study needs to be updated. |
built with Refined Cloudflare Pages Action⚡ Cloudflare Pages Deployment
|
Thank you Ben for the review @heysujal pls. commit the suggestions made by him and merge the latest changes with the main to your branch it will fix the failing workflow. |
Co-authored-by: Ben Hutton <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1176 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 10 10
Lines 373 373
Branches 94 94
=========================================
Hits 373 373 ☔ 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.
Thanks for your work on this!
I notice the Cookpad logo looks stretched.
The image itself isn't, so this is a CSS issue. It will need to be fixed =]
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.
Looks great. I'll approve as soon as the last changes are done.
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.
Good stuff. I just added the style required to fix it.
@benjagm Thanks a lot for the style fix. When I first made this PR, I noticed the issue and tried to fix it using other CSS property but couldn't make it work. |
What kind of change does this PR introduce?
This PR adds a new entry in
data/case-studies.json
for the missing Cookpad case studyIssue Number: #1159
Screenshots/videos:
Does this PR introduce a breaking change?
No