search

open-sauced / landing-page

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

feat: upgrade Sanity Studio to v3 #334

Closed brandonroberts closed 1 month ago

brandonroberts commented 1 month ago

Description

Related Tickets & Documents

Closes #328 Closes #333 Closes #335

Mobile & Desktop Screenshots/Recordings

Steps to QA

  1. checkout PR
  2. cd sanity
  3. Run npm install
  4. Run npm start
  5. Navigate to http://localhost:3333 to view studio

Tier (staff will fill in)

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

netlify[bot] commented 1 month ago

Deploy Preview for opensauced-landing ready!

Name Link
Latest commit b453605eca11c53c5be7e35b8f6538f672ae7905
Latest deploy log https://app.netlify.com/sites/opensauced-landing/deploys/66983ca9ed534b0008ed54b3
Deploy Preview https://deploy-preview-334--opensauced-landing.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

BekahHW commented 1 month ago

@brandonroberts it appears that Sanity Studio broke in one of my previous PRs. Idk if we want to address that here? I'm not able to run Sanity Studio on your branch either.

nickytonline commented 1 month ago

@brandonroberts it appears that Sanity Studio broke in one of my previous PRs. Idk if we want to address that here? I'm not able to run Sanity Studio on your branch either.

Consider fixing that in a separate PR @BekahHW as it's not related to the upgrade.

BekahHW commented 1 month ago

@nickytonline I think it is related to the upgrade, but not caused by this PR. That's why I mentioned it here.

brandonroberts commented 1 month ago

@BekahHW I ran into that also. If you run npm install at the root, then go into the sanity directory and run npm install, then npm start it works correctly. Not sure why that's needed as sanity is a dependency in both package.json files.