-
Notifications
You must be signed in to change notification settings - Fork 110
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
feat: add v1 node types constraints #279
Conversation
/** | ||
* Node types before `tables` implementation | ||
*/ | ||
export const V1_NODE_TYPES = [ |
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.
Question: should we add this as a schema constraint or node type? I thought this would be more suitable here by looking at the code.
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.
This is probably the most appropriate place for it even though it's not properly speaking a "schema" constraint. If anything I would push for the file name to be genericized to something like constraints.ts
(not important for this PR).
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.
Left a nit comment, but it's non-blocking, 👍
/** | ||
* Node types before `tables` implementation | ||
*/ | ||
export const V1_NODE_TYPES = [ |
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.
This is probably the most appropriate place for it even though it's not properly speaking a "schema" constraint. If anything I would push for the file name to be genericized to something like constraints.ts
(not important for this PR).
@@ -96,3 +97,29 @@ export const CONTAINERS = { | |||
[BLOCKS.TABLE_CELL]: [BLOCKS.PARAGRAPH], | |||
[BLOCKS.TABLE_HEADER_CELL]: [BLOCKS.PARAGRAPH], | |||
}; | |||
|
|||
/** | |||
* Node types before `tables` implementation |
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.
nit: "implementation" a bit technical, maybe "release"?
66dd9a3
to
db97672
Compare
No description provided.