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

MODELINKS-248: Extend authorities with Additional fields #318

Merged
merged 13 commits into from
Sep 6, 2024

Conversation

elena-shmygaliova
Copy link
Collaborator

@elena-shmygaliova elena-shmygaliova commented Aug 23, 2024

Purpose

Cataloguers or Librarians require highly precise tagging in the References section for Authority references.
To achieve this, an Advanced References Classification should be implemented for authority records.
This classification involves additional mappings to specific types of headings, namely "broaderTerm", "narrowerTerm", "earlierHeading", "laterHeading" for sft and saft headings.
The mod-entities-links module should be capable of processing this new set of fields, mapping them to entities for database storage, and transmitting them to other services via Kafka messages.

Changes Checklist

  • API Changes: extend AuthoritiesDto with the new fields
  • Unit Testing: Confirm that changed classes were covered by unit tests.
  • Integration Testing: Confirm that changed logic was covered by integration tests.
  • NEWS: Confirm that the NEWS file is updated with relevant information about the changes made in this pull request.

@elena-shmygaliova elena-shmygaliova force-pushed the feature/MODELINKS-248-extend-authorities branch from b152c52 to 2042146 Compare August 23, 2024 14:45
Copy link

sonarcloud bot commented Sep 6, 2024

@elena-shmygaliova elena-shmygaliova merged commit 55c50be into master Sep 6, 2024
6 checks passed
@elena-shmygaliova elena-shmygaliova deleted the feature/MODELINKS-248-extend-authorities branch September 6, 2024 10:42
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.

5 participants