Skip to content

MODFQMMGR-1090: Add open access entities#1191

Open
elsenhans wants to merge 11 commits intomasterfrom
MODFQMMGR-1090-add-oa-entities
Open

MODFQMMGR-1090: Add open access entities#1191
elsenhans wants to merge 11 commits intomasterfrom
MODFQMMGR-1090-add-oa-entities

Conversation

@elsenhans
Copy link
Contributor

https://folio-org.atlassian.net/browse/MODFQMMGR-1090

Purpose

Integrate Open Access entity types (complementing the types defined in):

  • Publication Status
  • Publication Identifier
  • Funding
  • Checklists
  • Correspondences
  • Open Access Journals (works)
  • Charges
  • Payers

See SQL-Statements in Jira issue for required fields and links.

Approach

  • add for every entity type a specification with the required sources and the columns
  • add translations for all colimns and entity types
  • add source-views for these tables: correspondence, funding, publication_identifier

TODOS and Open Questions

  • Use GitHub checklists. When solved, check the box and explain the answer.

Learning

Describe the research stage. Add links to blog posts, patterns, libraries or addons used to solve this problem.

Pre-Merge Checklist

If you are adding entity type(s), have you:

  • Added the JSON5 definition to the src/main/resources/entity-types directory?
  • Ensured that GETing the entity type at /entity-types/{id} works as expected?
  • Added translations for all fields, per the translation guidelines? (Check this by ensuring GET /entity-types/{id} does not have mod-fqm-manager.entityType. in the response)
  • Added views to liquibase, as applicable?
  • Added required interfaces to the module descriptor?
  • Checked that querying fields works correctly and all SQL is valid?

If you are changing/removing entity type(s), have you:

  • Added migration code for any changes?

@elsenhans elsenhans requested a review from a team as a code owner March 10, 2026 15:46
@elsenhans elsenhans requested a review from ncovercash March 13, 2026 09:12
@sonarqubecloud
Copy link

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.

1 participant