Repository navigation
Move sandboxed preview to quarto-dev.github.io/q2-sandbox (bd-73lilznv) - #919
Merged
Merged
Conversation
…-73lilznv) quarto-dev.github.io/q2 is being freed for the Q2 documentation. The sandboxed preview iframe now defaults to https://quarto-dev.github.io/q2-sandbox/, served from the gh-pages branch of the new quarto-dev/q2-sandbox repo. deploy-q2-sandbox.yml builds the bundle here and force-pushes it there with a deploy key scoped to that repo (secret Q2_SANDBOX_DEPLOY_KEY). deploy-sandboxed-preview.yml keeps publishing to /q2/ until production quarto-hub.com is rebuilt against the new default; it is then deleted. Plan: claude-notes/plans/2026-10-26-move-sandbox-free-q2-pages.md Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…lznv) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Contributor
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
The fenced code block inside a list item needs blank lines around it and must sit at the item's content column, and '~2027' is an unclosed subscript. Caught by the strict claude-notes render in CI. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
Today
https://quarto-dev.github.io/q2/serves theq2-sandboxed-previewiframe. We want that URL for the Q2 documentation. This PR moves the sandbox tohttps://quarto-dev.github.io/q2-sandbox/and leaves the old deploy running until production switches over.Plan:
claude-notes/plans/2026-10-26-move-sandbox-free-q2-pages.md(strand bd-73lilznv).What
deploy-q2-sandbox.yml. A workflow can't deploy Pages for another repo, so this one builds the sandbox bundle here and force-pushes it as a single commit to thegh-pagesbranch of the newquarto-dev/q2-sandboxrepo. It uses plaingitwith a deploy key that has write access toq2-sandboxonly, stored as theQ2_SANDBOX_DEPLOY_KEYsecret. There is no third-party action.VITE_Q2_SANDBOXED_PREVIEW_URL→https://quarto-dev.github.io/q2-sandbox/. The origin is unchanged, so the pinnedpostMessagetarget origin stays the same.assetPolicy.test.tsuses the new scope. A new test checks that/q2/(the docs) is outside the sandbox's scope even though/q2is a string prefix of/q2-sandbox.github-pages.md, the separate-origin design doc, the snyk KaTeX reference, and the hub-client changelog.deploy-sandboxed-preview.ymlis marked transitional and still deploys to/q2/. The quarto-hub.com build in production still points there until it is rebuilt.Already done outside this PR
quarto-dev/q2-sandbox, its deploy key, and the secret on this repo.gh-pagesby hand from a local build and turned on Pages. https://quarto-dev.github.io/q2-sandbox/ is live. The page,serviceWorker.js, the JS/CSS chunks and the KaTeX fonts all return 200 with the right content types.Testing
npm run test: 1686/1686 passed.npm run test:wasm: 434 passed, 28 skipped.tsc -bis clean.deploy-q2-sandbox.ymlonly triggers on pushes tomain, so CI on this PR doesn't exercise it. Its first run is on merge, and I'll check that it succeeds.After merge (separate steps)
quarto-hub-deploymentand check that the live bundle references/q2-sandbox/.deploy-sandboxed-preview.ymland add the docs deploy to/q2/. That deploy will include a self-unregisteringserviceWorker.jsso stale sandbox service workers at scope/q2/can't return 502s on docs pages.Out of scope: giving the sandbox an origin of its own instead of sharing
quarto-dev.github.io(bd-gotwa715).🤖 Generated with Claude Code