Skip to content
This repository has been archived by the owner on Jun 19, 2023. It is now read-only.

feat: added a new interop structure example #2

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

Giacomo-Papaluca
Copy link

No description provided.

@Giacomo-Papaluca Giacomo-Papaluca self-assigned this May 5, 2022
@Giacomo-Papaluca Giacomo-Papaluca marked this pull request as ready for review May 5, 2022 13:33
@@ -0,0 +1,18 @@
@prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Prefix declaration should end with .
Change declarations in all files

Suggested change
@prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#>
@prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#> .

interop:registeredAt "2020-04-04T20:15:47.000Z"^^xsd:dateTime ;
interop:updatedAt "2020-04-04T21:11:33.000Z"^^xsd:dateTime ;
interop:registeredAgent projectron:\#id ;
interop:hasAccessGrant alice-agents:2f2f3628\/27eae14b .
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Declaration jack-agents instead of alice-agents.
Happens in some other files so check other files and change there as well.

Suggested change
interop:hasAccessGrant alice-agents:2f2f3628\/27eae14b .
interop:hasAccessGrant jack-agents:2f2f3628\/27eae14b .

jack-agents:
a interop:AgentRegistry ;
interop:hasSocialAgentRegistration
alice-agents:c4562da9\/ , # Alice
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
alice-agents:c4562da9\/ , # Alice
alice-agents:c4562da9\/ ; # Alice

interop:hasSocialAgentRegistration
alice-agents:c4562da9\/ , # Alice
interop:hasApplicationRegistration
alice-agents:2f2f3628\/ , # Projectron
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
alice-agents:2f2f3628\/ , # Projectron
alice-agents:2f2f3628\/ . # Projectron

@@ -0,0 +1,15 @@
@prefix jack: <https://localhost:3000/jackpapa>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing some declarations

Suggested change
@prefix jack: <https://localhost:3000/jackpapa>
@prefix foaf: <http://xmlns.com/foaf/0.1/>.
@prefix solid: <http://www.w3.org/ns/solid/terms#>.
@prefix auth-agent: <http://localhost:4200/>.
@prefix interop: <http://www.w3.org/ns/solid/interop#>.
@prefix jack: <https://localhost:3000/jackpapa>.

@prefix jack-agents: <https://localhost:3000/jackpapa/agents/>
@prefix pm-shapetrees: <http://data.example/shapetrees/pm#>

alice-authorization:54a1b6a0
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
alice-authorization:54a1b6a0
jack-authorization:54a1b6a0

@@ -0,0 +1,15 @@
@prefix jack: <https://localhost:3000/jackpapa>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
@prefix jack: <https://localhost:3000/jackpapa>
@prefix foaf: <http://xmlns.com/foaf/0.1/>.
@prefix solid: <http://www.w3.org/ns/solid/terms#>.
@prefix auth-agent: <http://authz-agent.example>. # fake uri
@prefix interop: <http://www.w3.org/ns/solid/interop#>.
@prefix jack: <https://localhost:3000/jackpapa>.

@@ -0,0 +1,16 @@
@prefix interop: <http://www.w3.org/ns/solid/interop#>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same for projectron as with the social agents this will be in the pod of projectron. An application registration refers to this interop:application with interop:registeredAgent

jack:registries
a interop:RegistrySet;
interop:HasAgentRegistry jack:agents\/;
interop:hasAuthorizationRegistry jack:authorizations\/;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
interop:hasAuthorizationRegistry jack:authorizations\/;
interop:hasAuthorizationRegistry jack:authorizations\/.

@@ -0,0 +1,18 @@
@prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Prefix declaration should end with .

Change this in every file.

Useful tool to check turtle syntax: http://ttl.summerofcode.be/

Suggested change
@prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#>
@prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#>.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants