sanity-io / sanity-template-nextjs-landing-pages

A Sanity powered landing page builder in Next.js for https://sanity.io/create
https://www.sanity.io/create/?template=sanity-io%2Fsanity-template-nextjs-landing-pages
158 stars 54 forks source link

feat: add vercel deploy support & simplify the entire setup #66

Closed stipsan closed 2 years ago

stipsan commented 2 years ago

You can't test it with sanity/create before merging

But you can use this button until then:

Deploy with Vercel

When I tried running this earlier today this is what I saw: image

logs ![image](https://user-images.githubusercontent.com/81981/187043013-1bcbf583-661c-4506-ada0-1e1e3b16eca1.png)

At first it was tempting to just see if all the trouble was just a missing --legacy-peer-deps somewhere. But after I wasted a ton of time earlier this week trying to migrate a gatsby starter that way and ultimately failing I instead moved this starter to the same setup as I did for https://github.com/sanity-io/sanity-template-nextjs-blog-comments

I would've loved to have time to give the docs and readme some love. But for now priority 1 was to get it functional and automated. More automation makes it more realistic we'll be able to keep it up to date.

Quick list over changes that happened to https://github.com/sanity-io/sanity-template-nextjs-blog-comments earlier today, and what's proposed to https://github.com/sanity-io/sanity-template-nextjs-landing-pages in this PR:

kmelve commented 2 years ago

📝 Changed routes:

/[[...slug]]
Screenshot of /[[...slug]]
(view full size)

And 1 other route:

Commit 29f4df39c66e0dd8a3bca88aed2995f39d2b98a8 (https://template-nextjs-landing-pages-eawihiu7v.sanity.build).

benjaminsinger commented 2 years ago

Hi, noticed the same on this starter project. Slightly frustrating as this is pretty much the single project starter I need to get Sanity up and running for our use case.