-
Notifications
You must be signed in to change notification settings - Fork 171
Open
Description
Context
The blockGroup field is currently optional in the Deepnote file schema, but it's required for proper layout rendering on deepnote.com. This creates issues when files are missing this field.
Changes
Make blockGroup a required field in the Deepnote block schema (packages/blocks/src/deserialize-file/deepnote-file-schema.ts).
Why Temporary?
This is a temporary requirement to ensure all Deepnote files work correctly with the current layout system on deepnote.com. In the future, we plan to:
- Refactor the layout system to not depend on
blockGroupbeing present - Make
blockGroupoptional again once the dependency is removed
Implementation
- Changed
blockGroup: z.string().optional()toblockGroup: z.string()in the schema - Updated all test fixtures to include
blockGroupfield - All existing
.deepnotefiles in the repository already have this field, so no migration needed
Testing
- ✅ All tests pass (446 tests)
- ✅ TypeScript type checking passes
- ✅ All existing
.deepnotefiles in the repo validate successfully
Related
This change ensures consistency across all Deepnote files and prevents layout issues on deepnote.com until the platform can handle missing blockGroup fields gracefully.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels