Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clean up metadata issues #834

Merged
merged 1 commit into from
Sep 30, 2024
Merged

Clean up metadata issues #834

merged 1 commit into from
Sep 30, 2024

Conversation

laritakr
Copy link
Contributor

@laritakr laritakr commented Sep 30, 2024

Story

Metadata yaml files were audited and edited to:

  • Change dates to be single rather than multiple
  • Add multiple: false on form when term is not multiple... form defaults to multiple and term defaults to singular.
  • Remove unnecessary state & arkivo_checksum
  • remove dc_access_rights, which shares the same predicate in fedora with access_rights. (the shared predicate means they do the same thing, and the term is not accessed except in indexing.)

Refs

Expected Behavior Before Changes

Many works types could be created but not edited.
Metadata single or multiple on form did not match production.

Expected Behavior After Changes

Screenshots / Video

All work types and collection were tested, and can be both created and edited.

Screenshot 2024-09-30 at 4 09 38 PM

Notes

OER and ETD do not currently work. These come in via Hyku and are not used in Adventist, so they have not been adequately set up. At this point, they should be deactivated in all tenants to avoid any potential issues.

- Change dates to be single rather than multiple
- Add multiple: false on form when term is not multiple... form defaults to multiple and term defaults to singular.
- Remove unnecessary state & arkivo_checksum
- remove dc_access_rights, which shares the same predicate in fedora with access_rights.
(the shared predicate means they do the same thing, and the term is not accessed except in indexing.)
@laritakr laritakr merged commit 655254f into main Sep 30, 2024
6 checks passed
@laritakr laritakr deleted the metadata-cleanup branch September 30, 2024 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants