CredentialEngine / CredentialRegistry

Repository for development of the Credential Registry
Apache License 2.0
12 stars 10 forks source link

Update How Some Envelope URLs are Handled #688

Open jeannekitchens opened 8 months ago

jeannekitchens commented 8 months ago

When resources for competencies, pathway components, rubric criterion, concepts, and collection members are published, they are within the envelope for their related, Competency Framework, Pathway, Rubric, Concept Scheme, and Collections. Thereby, when a user uses an envelope URI for these classes, they get back, "Couldn't find Envelope."

https://credentialfinder.org/competency/ce-9d4c0fb0-42fd-4a03-8dfc-76890ff64c24 yields the following: { errors: [ "Couldn't find Envelope" ] }

Preferred change: Registry team request a change to get the envelope for the related Competency Framework, Pathway, Rubric, Concept Scheme, and Collections for:

siuc-nate commented 1 month ago

I think the intended URL is meant to be: https://credentialengineregistry.org/envelopes/ce-9d4c0fb0-42fd-4a03-8dfc-76890ff64c24

Currently, trying to load the /graph/ URI for a competency will return its framework's graph: https://credentialengineregistry.org/graph/ce-9d4c0fb0-42fd-4a03-8dfc-76890ff64c24

If that logic was extended a bit to hand back the envelope for the framework, I think that meets the ask here.

excelsior commented 3 weeks ago

@jeannekitchens @siuc-nate This feature is available on sandbox.

siuc-nate commented 3 weeks ago

Looks like this is working on sandbox, though it is also showing the envelope for bnode ids: https://sandbox.credentialengineregistry.org/ce-registry/envelopes/_:71dc7127-3e1a-46e7-8717-8375ef56807e

excelsior commented 1 week ago

@siuc-nate Bnode IDs are ignored now. If it looks good to you, I'll promote these changes to production.

siuc-nate commented 1 week ago

Looks good, thanks.