Skip to content

Add JSON Schema & XML DTD definitions of message data model #439

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

Merged
merged 3 commits into from
Aug 8, 2023

Conversation

eemeli
Copy link
Collaborator

@eemeli eemeli commented Jul 24, 2023

PR #393 left out the JSON Schema definition of the data model, which it originally referred to as its canonical description. This PR re-presents that, but now together with a synonymous message.dtd document. Unlike before, these definitions are here referenced not as canonical, but as equivalent to the TypeScript definitions.

The data model documents are also moved to spec/data-model/, to increase their distance slightly from the rest of the spec.

The intent with including these definitions is to increase the utility of the data model for applications that do have a need for representing messages as structured data rather than syntax. By providing ready definitions, we make it easier for such applications to naturally align for compatibility, without imposing any requirements that all implementations must comply with.

@aphillips aphillips added the data model Issues related to the Interchange Data Model label Jul 24, 2023
@eemeli
Copy link
Collaborator Author

eemeli commented Aug 8, 2023

@aphillips I resolved the conflicts on this, and this should now be good to merge. I'll update #444 after that's done, so that it too can be merged.

@aphillips aphillips merged commit 4f9f7b1 into unicode-org:main Aug 8, 2023
@eemeli eemeli deleted the data-more branch August 8, 2023 02:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data model Issues related to the Interchange Data Model
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants