Skip to content

Conversation

ryanbonial
Copy link
Member

@ryanbonial ryanbonial commented Sep 10, 2025

Description

This PR changes the Kitchensink test app to use projects that belong to the Sanity Sandbox organization so that it can be more easily tested and developed inside of the Sanity Dashboard. The main changes include:

  • Replaced the player document type with a movie document type in the multi-resource tests
  • Updated project IDs and organization IDs to use the Sanity Sandbox organization
  • Changed the dev command to use sanity dev instead of vite
  • Updated all references to the old project throughout the codebase
  • Modified the E2E tests to work with the new movie schema instead of player schema

What to review

  • The multi-resource route component now uses movie documents instead of player documents
  • Updated schema definitions and projections for the movie type
  • Configuration changes in sanity.config.ts, sanity.cli.ts, and package.json
  • E2E test modifications to work with the new document type

Testing

The E2E tests have been updated to work with the new document types and should pass with the new configuration.

Fun gif

pivot

Copy link

vercel bot commented Sep 10, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
sdk-docs Ready Ready Preview Comment Sep 10, 2025 8:02pm
sdk-kitchensink-react Ready Ready Preview Comment Sep 10, 2025 8:02pm

Copy link
Member

@cngonzalez cngonzalez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Should we get rid of v28v5k8m entirely?

@ryanbonial
Copy link
Member Author

Thanks. No, we should not get rid of that project it is used for https://sdk-explorer.sanity.io

@ryanbonial ryanbonial merged commit 2b99c19 into main Sep 11, 2025
19 checks passed
@ryanbonial ryanbonial deleted the rb/move-kitechensink-sandbox branch September 11, 2025 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants