kadaster-labs / solid-quest

Our journey on SOLID
https://labs.kadaster.nl/cases/Solid-Pods
MIT License
3 stars 1 forks source link

Feat/zkp #71

Closed kad-busses closed 1 year ago

kad-busses commented 1 year ago

Note: had to create a patch for one of the deps, will create a separate PR to said repo

marcvanandel commented 1 year ago

En misschien nog de .gitignore uitbreiden met de volgende files (want worden automagisch gegenereerd als ze niet bestaan, toch?):

solid-pod-provider/data/verkoper-vera/private/
solid-pod-provider/data/verkoper-vera/public/

En de vc-api verwijderen uit start-backing-services.sh 😉

kad-busses commented 1 year ago

Dat van die .gitignore is wel een goeie ja, we kunnen wel kijken wat we minimaal ingecheckt willen hebben en de rest ignoren. Als je nu een nieuw account aanmaakt (bijv. "tester-timo"), dan komen al diens files ook onder versiebeheer.

De WebIDs van koper-koos en verkoper-vera zijn nu nog wel hardcoded op plekken, dus die moeten voor nu nog wel blijven. En de ACLs op een aantal folders zijn nu ook nodig. Verder eigenlijk niet, denk ik? Of vind je het wel prettig om ergens een voorbeeld koopovereenkomst (en de events) te hebben staan, zoals bijvoorbeeld dit event?

marcvanandel commented 1 year ago

Ik heb 345 telkens laten staan en zelfs bijgewerkt ... om een voorbeeld in de code te hebben zitten. Die mag van mij wel blijven ... hoewel dat niet echt meer nodig is. Tegelijk is het wel gemakkelijk om daar naar te verwijzen als we later een voorbeeld nodig hebben, niet? Ik ga vast niet altijd een werkende omgeving paraat hebben (hoe mooi ons onderzoek / demonstrator ook is 😁 )