Skip to content
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.

Sign up for GitHub

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

Jump to bottom

docs(examples): use vercel integration in cms-sanity #39323

Merged
merged 3 commits into from Aug 12, 2022
Merged

docs(examples): use vercel integration in cms-sanity #39323

merged 3 commits into from Aug 12, 2022

Conversation

stipsan
Copy link
Contributor

@stipsan stipsan commented Aug 4, 2022 •
edited

Feature

  • Documentation added

Documentation / Examples

This PR updates the cms-sanity example to:

  • Deploy to Vercel with the Sanity Vercel Integration by default.
  • You can still opt-out of using the Sanity Vercel Integration, clicking You can also set up manually gives you all the information you need.
  • The blog itself is updated so it's much more resilient to missing data, and is setup to set revalidate: 60 until you have a On-demand Revalidation webhook setup.
  • Preview Mode is now enabled on the frontpage as well.
  • The Sanity client setup, and webhook validation, are updated to follow our current best practices.

@ijjk ijjk added the examples Issue/PR related to examples label Aug 4, 2022
@stipsan stipsan force-pushed the upgrade-cms-sanity branch 6 times, most recently from 7c8f1bc to 36cb890 Compare August 9, 2022 10:58
@stipsan stipsan marked this pull request as ready for review August 9, 2022 12:22
@stipsan stipsan force-pushed the upgrade-cms-sanity branch 15 times, most recently from 0f5e2b4 to c5f5357 Compare August 11, 2022 17:43
leerob
leerob previously approved these changes Aug 12, 2022
View reviewed changes
@kodiakhq kodiakhq bot merged commit 0c7217b into vercel:canary Aug 12, 2022
@stipsan stipsan deleted the upgrade-cms-sanity branch August 12, 2022 04:36
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ijjk ijjk ijjk approved these changes

@leerob leerob leerob left review comments

@timneutkens timneutkens Awaiting requested review from timneutkens timneutkens is a code owner

@shuding shuding Awaiting requested review from shuding shuding is a code owner

@styfle styfle Awaiting requested review from styfle styfle is a code owner

@steven-tey steven-tey Awaiting requested review from steven-tey

@balazsorban44 balazsorban44 Awaiting requested review from balazsorban44

Assignees
No one assigned
Labels
examples Issue/PR related to examples
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@stipsan @leerob @ijjk