terraframe / geoprism-registry

GeoPrism Registry is a system for curating interlinked data through time. It's the first framework implementing the Common Geo-Registry specification.
https://geoprismregistry.com/
GNU Lesser General Public License v3.0
18 stars 5 forks source link

Issue #930 Proposed Solution #934

Closed shalul closed 4 months ago

shalul commented 4 months ago

Hello,

We were looking through the issues and found https://github.com/terraframe/geoprism-registry/issues/930 on metadata fields, and here is our proposed solution.

Solution

Commenting out the the createRow for Label and Code so it doesn't show on the spreadsheet. In addition, there seems to be a getValue missing from the LISTPROGRESS attribute, and we speculate that that might've been why it wasn't showing up on the spreadsheet. To solve this problem, we have included that at the end.

Alternative solutions

We believe this is the only solution, as this seems to be a display problem as well. We are unsure about future solutions.

Follow-Up Questions

In terms of the other questions, we were confused on how to implement the acknowledgement fields and the order on the GPR due to lack of information. If you could please provide more information on these two issues, that would be wonderful!

Thanks in advance!

terraframe-builder commented 4 months ago

Can one of the admins verify this patch?

rrowlands commented 4 months ago

ok to test

rrowlands commented 4 months ago

add to whitelist

rrowlands commented 4 months ago

Thanks so much for your commit! I looked this over and it seems to be good to go. If we have time in our next sprint to look more into the ticket we'll let you know.