scientist-softserv / adventist_knapsack

Apache License 2.0
1 stars 0 forks source link

Metadata fields not showing on Work public view #242

Closed KatharineV closed 7 months ago

KatharineV commented 8 months ago

Some metadata fields can have content entered on the edit screen, but when I save, the content is not present on the user view. The fields don't show up at all. Some of these are new metadata fields in Knapsack that I can't test against our existing Hyku. Some of the fields are in the older version and I can confirm that they show on ADL prod but not in Knapsack.

New Fields:

Screenshot of edit screen of this work.

Image

Existing fields:

Work types affected:

Testing Instructions

Create a work of each work type, filling in each term. Ensure that each term appears on the show view for the work.

Terms by work type The expected terms for each type of work are listed below, along with their sequences on the form and show pages. ![adv metadata by work type ](https://github.com/scientist-softserv/adventist-dl/assets/17851674/a349364a-71fa-4f9a-bae4-e98b911c8bc7)
Screenshots showing new terms ![Screenshot 2023-10-16 at 7 50 59 PM](https://github.com/scientist-softserv/adventist-dl/assets/17851674/d4594a35-f503-4394-a083-18c2b18e1c7a) ![Screenshot 2023-10-16 at 8 08 34 PM](https://github.com/scientist-softserv/adventist-dl/assets/17851674/dda51499-9fdd-4ca 9-87ad-7943d1250dab)
laritakr commented 8 months ago

You are referring to some existing terms that your current app has specifically from being shown on the works: Conference Items, Datasets, Journal Articles, Published Works and Theses exclude description and license (they show abstract) Exam Papers exclude license Generic Works and Images should show all terms

Rights Notes and Access Rights are new terms for all work types, but they were for some reason excluded from Journal Articles. They can either be excluded from other work types, or made to appear on the show view - do you have a preference?

KatharineV commented 8 months ago

@laritakr I may not be understanding correctly, so please bear with me. I didn’t know that my predecessor had the team suppress any fields from any work types. There is no reason I can think of for why we would suppress any fields. So, I would ask for all new fields to appear in our version of Hyku on every work type where they are intended to appear. This applies to Rights Notes and Access Rights, I understand.

As for the other fields, forget I mentioned them I guess. My predecessor kept literally no documentation of decisions he made like fields to suppress, so I had no way of knowing what to look for. My apologies for wasting your time. It is very difficult to do this testing in the dark. I am doing my best.

laritakr commented 8 months ago

No worries, Katharine... this is confusing for both of us!

I am doing some verification and we're trying to do some cleanup in the code to make this more straightforward for the future. I put together a spreadsheet where I am verifying the metadata, so I may have more questions next week. I appreciate your patience as I try to figure it out. :) But unless you have a reason to suppress any fields from showing, I intend to have them show.

laritakr commented 8 months ago

I will add that publication status is DEFINITELY a bug... it is not persisting on the work so it does not display. I will continue working to get this resolved.

KatharineV commented 8 months ago

Thanks, LaRita. I agree that there's no reason to suppress any fields. If we don't enter metadata on the backend, they don't show on the front, so that's a fine solution. I can't think of any reason to redact a field from the backend, thereby making it inaccessible for future use, as if I know all the possible future use cases for our instance of Hyku. I don't have that much hubris. So, yes, please do make anything show that is available, and I'll plan for a future maintenance ticket to release the fields that were suppressed in earlier rounds of work under a different mindset.

Thank you also for catching publication status. That one is weird!

kirkkwang commented 7 months ago

QA Results

Notes

These are non-blocking but worth noting:

I couldn't test the Location probably because GeoName isn't set up on staging.

I'm noticing an in consistency with the date fields

image

Sometimes it's doing MM/DD/YYYY and sometimes DD/MM/YYYY, first one should be November 1st and second one is November 9th.

kirkkwang commented 7 months ago

with regards to the date appearing weird, this PR seems to address it

KatharineV commented 7 months ago

Fields are showing as expected on Knapsack Demo. I created all new works of every type today, and I filled in every field. With the exception of those that were intentionally suppressed in the past, all fields show on the user side now.