open-sauced / landing-page

This is the new landing page for open sauced
https://opensauced.pizza
32 stars 29 forks source link

feat: `/students` marketing page #223

Closed zeucapua closed 7 months ago

zeucapua commented 7 months ago

Description

Implements /students page with data from Sanity CMS.

What type of PR is this? (check all applicable)

Related Tickets & Documents

Closes #222

Mobile & Desktop Screenshots/Recordings

Steps to QA

Added to documentation?

[optional] Are there any post-deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?

netlify[bot] commented 7 months ago

Deploy Preview for opensauced-landing ready!

Name Link
Latest commit f39f77da82b5063a71a0b11ad356cbb5a89fc7e6
Latest deploy log https://app.netlify.com/sites/opensauced-landing/deploys/65f9bda4730ed100086bdac6
Deploy Preview https://deploy-preview-223--opensauced-landing.netlify.app
Preview on mobile
Toggle QR Code...

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: 82 (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.

isabensusan commented 7 months ago

⚠️ Let's not merge the PR just yet, I need to finish setting up the images

The changes look good, but for some reason the image coming from Sanity's CDN aren't loading. Not sure if this is just a deployment preview issue though. cc: @BekahHW

CleanShot 2024-03-19 at 12 56 40

It's loading fine! this is just a placeholder for the images :)

also, LGTM 💯