Closed bdougie closed 3 months ago
Name | Link |
---|---|
Latest commit | 3f0289a2512fcd5e8d2f60cdc0cf45a22f3e8ff3 |
Latest deploy log | https://app.netlify.com/sites/opensauced-landing/deploys/663237fa89fb6a0008a6cfe0 |
Deploy Preview | https://deploy-preview-257--opensauced-landing.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
Lighthouse |
1 paths audited Performance: 100 (no change from production) Accessibility: 91 (no change from production) Best Practices: 92 (no change from production) SEO: 83 (no change from production) PWA: - View the detailed breakdown and full score reports |
To edit notification comments on pull requests, go to your Netlify site configuration.
Description
nick noticed there was an eslint error on #254. I took a look and found the error to unrelated to that PR and fixed it on a new PR.
There is an
img
that should of beenImage
and was also missing aria labels.Related Tickets & Documents
closes #259
Mobile & Desktop Screenshots/Recordings
Steps to QA
Run
npm run build
locally to see it succeed.Tier (staff will fill in)
[optional] Are there any post-deployment tasks we need to perform?
More errors exist that are Next warnings. I opened an issue for that. https://github.com/open-sauced/landing-page/issues/258
[optional] What gif best describes this PR or how it makes you feel?