Skip to content

test(component): condensed notation will overwrite schema#571

Closed
redgeoff wants to merge 2 commits intomainfrom
condensed-with-schemas
Closed

test(component): condensed notation will overwrite schema#571
redgeoff wants to merge 2 commits intomainfrom
condensed-with-schemas

Conversation

@redgeoff
Copy link
Copy Markdown
Owner

@redgeoff redgeoff commented Jan 17, 2022

The condensed notation support introduced by #569, overwrites the schema when a derived component specifies a schema. Ultimately, this probably means that we cannot support this notation.

Specifically, we need https://github.com/redgeoff/mson/pull/571/files#diff-63bd7813ac255f841a13b8f1405bd3d8065aa00be44f49b77021df1e5128ff04R80 and NOT https://github.com/redgeoff/mson/pull/571/files#diff-63bd7813ac255f841a13b8f1405bd3d8065aa00be44f49b77021df1e5128ff04R99

@redgeoff
Copy link
Copy Markdown
Owner Author

For now, opting not to support a className property. Instead, it has been documented as a possible future in enhancement by #574

@redgeoff redgeoff closed this Jan 18, 2022
@redgeoff redgeoff deleted the condensed-with-schemas branch January 18, 2022 00:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant