-
Notifications
You must be signed in to change notification settings - Fork 87
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
Change collection name to collection id #171
Comments
Yes, it will be |
That's great it's going from {name} to {collectionId} throughout. But the issue we're having is actually in the response document, not in the URL template. In example 4 it's:
That first 'name' there is what we'd like to be 'id'. That would also be consistent with the URL templates, etc. Are you planning to change that too? Would be really great, and would make sense for the response document to clearly identify that from {collectionId} you get a document that says the id. |
@cholmes - Changing it to |
I support that. I actually thought changing the URL template would imply also changing the response, otherwise it would be even more inconsistent than before. |
"name", "id", "collectionId" ... all good with me! |
Awesome. |
It also is more obvious when a JSON-LD context declares that property to be the node identifier "@id" ;-) See https://www.w3.org/TR/json-ld/#node-identifiers |
+1 for changing |
+1 for changing name to id. |
+2 |
In the specification it is defined that each collection has a
name
. This name is expected to be a unique identifier. I'd strongly vote to change this to a collectionid
due to the following reasons:This was also shortly proposed in the comment by @cholmes here: #156 (comment)
This is also discussed in other projects like STAC (radiantearth/stac-spec#262) and openEO, which all tend to like id and title more and want to omit name because its ambiguous.
The text was updated successfully, but these errors were encountered: