-
Notifications
You must be signed in to change notification settings - Fork 2
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
Fit into SAI #4
Fit into SAI #4
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|
|
@@ -11,3 +11,14 @@ In the project directory, you can run: | |||||||||||
npm install | ||||||||||||
npm run start | ||||||||||||
``` | ||||||||||||
|
||||||||||||
## Add this app to your pod using SAI | ||||||||||||
In order to add this app to your pod: | ||||||||||||
* use [this branch of SAI](https://github.com/michielbdejong/sai-js/tree/investigating-77), and run it on localhost according to the instructions there | ||||||||||||
* run this app according to the instructions above | ||||||||||||
* go to http://localhost:4200/applications | ||||||||||||
* click '+ ADD APPLICATION' | ||||||||||||
* fill in http://localhost:3600 as the Client ID | ||||||||||||
Comment on lines
+19
to
+21
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
you can find out how to do it at https://github.com/janeirodigital/sai-js/blob/main/examples/vuejectron/src/store/core.ts#L53-L59 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Those instructions will fail unless you also add that code into this app, right? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It does work; you still need a valid Client ID document janeirodigital/sai-js#77 (comment) I still think requesting authorization from the app is more intuitive than making sure one passes a valid client ID into the input field. But since we are currently talking about app developers, it should not matter. |
||||||||||||
* click Next | ||||||||||||
* grant access | ||||||||||||
* launch the app |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,33 @@ | ||
PREFIX xsd: <http://www.w3.org/2001/XMLSchema#> | ||
PREFIX rdfs: <http://www.w3.org/2000/01/rdf-schema#> | ||
PREFIX interop: <http://www.w3.org/ns/solid/interop#> | ||
PREFIX acl: <http://www.w3.org/ns/auth/acl#> | ||
PREFIX shape | ||
|
||
<#this> | ||
a interop:AccessNeedGroup ; | ||
interop:accessNecessity interop:accessRequired ; | ||
interop:accessScenario interop:PersonalAccess ; | ||
interop:authenticatesAs interop:SocialAgent ; | ||
interop:hasAccessDescriptionSet | ||
<./descriptions-en> , | ||
<./descriptions-es> ; | ||
interop:hasAccessNeed <#need-foaf-profile> . | ||
|
||
<#need-foaf-profile> | ||
a interop:AccessNeed ; | ||
interop:registeredShapeTree <#profileTree> ; | ||
interop:accessNecessity interop:AccessRequired ; | ||
interop:accessMode acl:Read, acl:Create, acl:Update, acl:Delete . | ||
|
||
<#profileTree> | ||
a shapetrees:ShapeTree ; | ||
shapetrees:expectsType shapetrees:Resource ; | ||
shapetrees:shape <./foafProfile.txt> ; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ShExC syntax registers |
||
shapetrees:describesInstance rdfs:label . | ||
|
||
<./descriptions-en> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You still need to add those descriptions; I think you should focus on English and remove Spanish below. |
||
interop:usesLanguage "en"^^xsd:language . | ||
|
||
<./descriptions-es> | ||
interop:usesLanguage "es"^^xsd:language . |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,15 @@ | ||
PREFIX ex: <https://example.com/> | ||
PREFIX foaf: <http://xmlns.com/foaf/0.1/> | ||
PREFIX rdfs: <http://www.w3.org/2000/01/rdf-schema#> | ||
PREFIX xsd: <http://www.w3.org/2001/XMLSchema#> | ||
|
||
ex:FoafProfile EXTRA a { | ||
a [ foaf:Person ] | ||
// rdfs:comment "Defines the node as a Person (from foaf)" ; | ||
foaf:name xsd:string ? | ||
// rdfs:comment "Define a person's name." ; | ||
foaf:img xsd:string ? | ||
// rdfs:comment "Photo link but in string form" ; | ||
foaf:knows @ex:FoafProfile * | ||
// rdfs:comment "A list of WebIds for all the people this user knows." ; | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,15 @@ | ||
{ | ||
"@context": [ | ||
"https://www.w3.org/ns/solid/oidc-context.jsonld", | ||
{ | ||
"interop": "http://www.w3.org/ns/solid/interop#" | ||
} | ||
] , | ||
"client_id": "http://localhost:3006/id.json", | ||
"client_name": "LDO React Tutorial", | ||
"logo_uri": "https://robohash.org/ldo-react-tutorial?set=set3", | ||
"redirect_uris": ["http://localhost:3006/callback"], | ||
"grant_types" : ["refresh_token","authorization_code"], | ||
"interop:hasAccessNeedGroup": "http://localhost:3006/access-needs.ttl", | ||
"interop:hasAuthorizationCallbackEndpoint": "http://localhost:4500" | ||
} |
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.
For the people to request authorization you will need to remove the ldo-react-tutorial one you added.
You will also need to add at least one data registry using your shapetree, for example here https://github.com/janeirodigital/sai-js/tree/main/packages/css-storage-fixture/alice-home/dataRegistry
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.
What happens if Alice uses this app for the first time and doesn't have the data registry it needs yet?
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.
-> janeirodigital/sai-js#80