-
Notifications
You must be signed in to change notification settings - Fork 499
fix(cli): handle url imports for themer.sanity.build #11346
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.
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
base: main
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
3 Skipped Deployments
|
🧪 E2E Preview environment🔑 Environment Variables for Local TestingThis is the preview URL for the E2E tests: https://e2e-studio-lk5dfen03.sanity.dev To run the E2E tests locally, you can use the following environment variables, then run 💬 Remember to build the project first with |
📊 Playwright Test ReportThis report contains test results, including videos of failing tests. |
⚡️ Editor Performance ReportUpdated Tue, 02 Dec 2025 12:44:53 GMT
Detailed information🏠 Reference resultThe performance result of
🧪 Experiment resultThe performance result of this branch
📚 Glossary
|
⚡️ Editor Performance ReportDeploying studio and running performance tests… |
Coverage Report
File CoverageNo changed files found. |
018fcf3 to
2f3de31
Compare
48af160 to
20cea3b
Compare
bjoerge
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Cool. I need an approval to be able to merge :) (running the eFPS tests again before merging ofc.. they failed as per usual) |
20cea3b to
a5ac793
Compare
Description
The PR adds an override for imports starting with
https://themer.sanity.build/api/that returns aProxyobject. This should be safe since the themes from the themer never impacts the exported schema.This was done in response to https://linear.app/sanity/issue/CLDX-2705/validateextract-schema-fails-when-there-is-a-url-import-in reported by a user in #7602.
Testing
Tests have been added both for the
importErrorHandlerin isolation and a test verifying that the themer url imports work in a worker where themockBrowserEnvironmentis called. Has also been verified in a test project locally.Notes for release
sanity.config.ts#7602)