Description
Follow-up to #1333 / #1335.
getData in blocks/sheet/utils/index.js skips emitting sheet-load-status for version-preview loads (versionId set) — intentionally, since firing it would incorrectly toggle the main sheet's grid/not-permitted banner over a historical-version fetch. But that means a 401/403 loading a specific version currently falls back to getDefaultSheet() silently: da-version-review (blocks/sheet/da-version-review.js) has no error handling at all and just renders the empty result as a blank table.
Expected
A failed version load should surface a "Sign in required" / "Not permitted" message in the version-preview panel itself, not a blank table.
🤖 Generated with Claude Code
Description
Follow-up to #1333 / #1335.
getDatainblocks/sheet/utils/index.jsskips emittingsheet-load-statusfor version-preview loads (versionIdset) — intentionally, since firing it would incorrectly toggle the main sheet's grid/not-permitted banner over a historical-version fetch. But that means a 401/403 loading a specific version currently falls back togetDefaultSheet()silently:da-version-review(blocks/sheet/da-version-review.js) has no error handling at all and just renders the empty result as a blank table.Expected
A failed version load should surface a "Sign in required" / "Not permitted" message in the version-preview panel itself, not a blank table.
🤖 Generated with Claude Code