regen-network / regen-registry-standards

:seedling: RDF and SHACL schemas for Regen Registry
4 stars 1 forks source link

feat: cfc metadata #24

Closed wgwz closed 1 year ago

wgwz commented 1 year ago

Description

Closes: https://github.com/regen-network/regen-registry/issues/1121


Author Checklist

All items are required. Please add a note to the item if the item is not applicable and please add links to any relevant follow up issues.

I have...

Reviewers Checklist

All items are required. Please add a note if the item is not applicable and please add your handle next to the items reviewed if you only reviewed selected items.

I have...

wgwz commented 1 year ago

@S4mmyb @mhagel left some comments to guide through the changes.

the most important files to review are the actual metadata:

then the shacl/ and the jsonld/ templates (see those folders for changes related to C02).

NOTE: i didn't write the shacl template for the C02 projects yet @S4mmyb since you mentioned that we're going to use the UI to create those and there was a big of ambiguity there.

wgwz commented 1 year ago

this is just a note that we might be able to use this method for sort of "inheriting" certain properties of other shapes:

https://book.validatingrdf.com/bookHtml011.html#sec126

/cc @mhagel

S4mmyb commented 1 year ago

Hey @wgwz , thanks for the PR! I requested a few changes, mostly in the templated docs. For the most part it looks good - I think one thing we want to make sure of is that when the the user flow to create a project page is being used it correctly fills out the fields in the json-ld being populated behind the scenes. This includes:

I also noticed that we don't have in here a section for media or project photos. I'm sure this is being automatically populated in the user-interface, we just need to make sure to include it in the schema validation in the same way we do for C01 project pages.

Overall great job!

wgwz commented 1 year ago

I also noticed that we don't have in here a section for media or project photos. I'm sure this is being automatically populated in the user-interface, we just need to make sure to include it in the schema validation in the same way we do for C01 project pages.

@S4mmyb we have previewPhoto and galleryPhotos fields (same fields as C01) for both of the two projects i've written the metadata for so far.

  "regen:previewPhoto": "https://regen-registry.s3.amazonaws.com/projects/C02/harvey-manning-01.jpg",
  "regen:galleryPhotos": [
    "https://regen-registry.s3.amazonaws.com/projects/C02/harvey-manning-02.jpg",
    "https://regen-registry.s3.amazonaws.com/projects/C02/harvey-manning-03.jpg"
  ],
wgwz commented 1 year ago

todo: remove the regen:cfcProjectId

S4mmyb commented 1 year ago

@wgwz I just gave this a final review and it all looks good! Changes/updates I made:

I'll go ahead and approve this, but before we merge we just need to make sure to add in the project start/end date to the King County project (C01-001) which is missing from the project design document. I emailed the CFC team to get those dates - once we have them all we need to do is fill them in and make sure it successfully conforms to the shape as it's currently failing on invalid date format (ie we don't have a date).

Once again, great work!

S4mmyb commented 1 year ago

Added in the final dates for the final C02 project C02-001 and all checks on the shacl-validate have passed so approving now cc @wgwz