LD4P / sinopia_editor

Sinopia Linked Data Editor
https://sinopia.io/
Apache License 2.0
35 stars 10 forks source link

Editing Template Doesn't Save Particular Lookup #2738

Closed jermnelson closed 3 years ago

jermnelson commented 3 years ago

Describe the bug What went wrong? What did you expect to happen? When editing the https://api.stage.sinopia.io/resource/UChicago:RT:bf2:Monograph:Dissertation template, the http://id.loc.gov/ontologies/bibframe/AdminMetadata's propertyTemplate is a nested resource property type. Searching (or pasting in the URL) for https://api.stage.sinopia.io/resource/UChicago_RT_BF2_AdminMetadata as the nested resource works correctly and can be selected. Screen Shot 2020-12-07 at 9 20 02 AM

Saving the template works as expected.

Doing a hard refresh and opening the UChicago:RT:bf2:Monograph:Dissertation, the UChicago_RT_BF2_AdminMetadata is not appearing as being saved:

Screen Shot 2020-12-07 at 9 22 46 AM

However in the API, https://api.stage.sinopia.io/resource/UChicago:RT:bf2:Monograph:Dissertation does have the UChicago_RT_BF2_AdminMetadata` persisted correctly in the JSON-LD.

Another wrinkle, is that if a different Template is selected for the AdminMetadata nested resource lookup, the Editor does display the correct resource template.

justinlittman commented 3 years ago

It is unhappy with the id UChicago_RT_BF2_AdminMetadata, which is not a URI.

If you create a new template with an id like UChicago:RT:BF2:AdminMetadata, it works correctly.

If this is an acceptable work-around, I will postpone further work on this until the next workcycle.

jermnelson commented 3 years ago

@michelleif, could you contact Jesse at University of Chicago and see if this is an acceptable work-around?

michelleif commented 3 years ago

@jyssy will this work?

jyssy commented 3 years ago

big thanks @jermnelson @justinlittman and @michelleif This seems to be working now, things are saving and staying saved. This is funny though that I created the ADMIN metadata in that strange style...I just need to make this change in production next. cheerz

michelleif commented 3 years ago

@jyssy can we close this?

jyssy commented 3 years ago

yes indeed. thank you