-
Notifications
You must be signed in to change notification settings - Fork 1
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
Proposal of POD Management #21
Conversation
This is a proposal of a possible management of the POD for the interoperability of the Viade apps.
I didn't add it to the index file, if I should do so, please tell me before merging it. |
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.
I think it would be easier to track this pull request if you create one or more issues first to discuss it.
Reading your proposal, I can see several issues that could be separately discussed:
- Using JSON-LD to represent the routes
- The folder structure
- How to handle the GPX information when routes are represented as JSON-LD
My group (es2a) has a few suggestions:
|
Thanks for your comments. As I said before, I think it will be easier to discuss all those points separately with different issues and maybe once they are solved, we could improve this pull request taking into account the agreed resolutions...or using different pull requests for each of them. |
Added elevation Added waypoints Added author Added new comment approach using an extra file.
It didn't find the json example.
Rdf namespace (http://www.w3.org/2000/01/rdf-schema#) is included in the context but is never used in route files. Same for xml shema (http://www.w3.org/2001/XMLSchema#) and viade:comment in comments jsonld. Is there a reason for that? Or is just an error? |
It is not necessary since as you say is not used in the context, I included it because of two reasons. The first one is that the proposal of @labra used that namespaces, I wanted to show that we can use them too. The second one is that in case that someone wanted to propose a new property, they can easily check in one of those namespaces if such a property already exists. In the end, I checked myself the documents to see if some properties matched. In fact I put exactly the same name in the comment example as the properties in Schema.org, you can see there the namespaces even though it is not used. In case no more is added to that file, I think that using an already defined property is a better approach. |
Formatting details: note that in bikeshed the final octothorpes are to be used only if the title has an ID, otherwise they appear in the final text. I added IDs to the titles so this doesn't happen and they can be referenced later if needed. Also, when building locally the json-ld route example is imported but appears as plain text without formatting. I was not able to solve this issue. |
By using this approach we ensure a high performance and an immense speedup with respect to the previous one.
I noticed that we are missing a specification for notifications to the inbox folder; I made one as a simplified version of the one proposed on #34 (comment) in jsonld format. I am no expert with these, so take this proposal with care.
We could add this to the proposal so we have a reference to make a base format for interoperability. |
Given that several people have said me that they agree with this pull-request, I proceed to merge it. |
This is a proposal of a possible management of the POD of the Viade apps. From the group en2a we expect that this proposal can be taken into account by other groups in order to achieve the interoperability beetwen our apps.