Add content types for NEST Desktop #275
Conversation
after discussion with Sebastian Spreizer
| }, | ||
| "@id": "https://openminds.om-i.org/instances/contentTypes/application_vnd.nest-desktop.model.v4+json", | ||
| "@type": "https://openminds.om-i.org/types/ContentType", | ||
| "dataType": null, |
There was a problem hiding this comment.
I mean, yes, because it's a JSON document, but semantically it's a configuration.
There was a problem hiding this comment.
there can be more than one data type. let's now not integrate this, but I'd like this field being filled for all content types in the long run
| }, | ||
| "@id": "https://openminds.om-i.org/instances/contentTypes/application_vnd.nest-desktop.model.v4+json", | ||
| "@type": "https://openminds.om-i.org/types/ContentType", | ||
| "dataType": null, |
instances/latest/contentTypes/application_vnd.nest-desktop.model.v4+json.jsonld
Outdated
Show resolved
Hide resolved
| "fileExtension": [ | ||
| ".json" | ||
| ], | ||
| "name": "application/vnd.nest-desktop.model.v4+json", |
There was a problem hiding this comment.
Should the version not come before ?
vnd.nest-desktop.v4.model+json
There was a problem hiding this comment.
I thought about that, but v4 of a format specification doesn't necessarily correspond to v4 of the software (in this case it does, but as a general rule I think the version spec should come at the end (just before any "+" modifiers)
There was a problem hiding this comment.
okay. we may revisit the existing content types. I don't think this was done consistently as you suggested (but I'm totally okay with that suggestion)
instances/latest/contentTypes/application_vnd.nest-desktop.model.v4+json.jsonld
Outdated
Show resolved
Hide resolved
instances/latest/contentTypes/application_vnd.nest-desktop.project.v4+json.jsonld
Outdated
Show resolved
Hide resolved
apdavison
added
the
ContentType
instances/latest/contentTypes/application_vnd.nest-desktop.model.v4+json.jsonld
Outdated
Show resolved
Hide resolved
…nd.nest-desktop.model.v4+json.jsonld Co-authored-by: Lyuba Zehl <[email protected]>
|
ready to merge, I think |
after discussion with Sebastian Spreizer