Skip to content

MODEXPW-630 eholdings export: Add additional export fields#744

Merged
viacheslavkol merged 3 commits into
masterfrom
MODEXPW-630
Jul 10, 2026
Merged

MODEXPW-630 eholdings export: Add additional export fields#744
viacheslavkol merged 3 commits into
masterfrom
MODEXPW-630

Conversation

@viacheslavkol

@viacheslavkol viacheslavkol commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Purpose

Add additional eholdings export fields:

  • packageDisplayName;
  • managedAlternativeNames;
  • customAlternativeNames;
  • managedDescription;
  • customDescription;
  • packageAccess;
  • hideInPublicationFinder;
  • hideInFullTextFinder;
  • excludeFromMARCExport;
  • titleProxiedUrl

Remove eholdings package fields:

  • packageShowToPatrons

MODEXPW-630
MODEXPW-501

Pre-Merge Checklist:

Before merging this PR, please go through the following list and take appropriate actions.

  • Does this PR meet or exceed the expected quality standards?
    • Code coverage on new code is 80% or greater
    • Duplications on new code is 3% or less
    • Check logging
    • There are no major code smells or security issues
  • Does this introduce breaking changes?
    • Were any API paths or methods changed, added or removed?
    • Were there any schema changes?
    • Did any of the interface versions change?
    • Were permissions changed, added, or removed?
    • Are there new interface dependencies?
    • There are no breaking changes in this PR.

@viacheslavkol viacheslavkol self-assigned this Jul 9, 2026
@sonarqubecloud

Copy link
Copy Markdown

@viacheslavkol viacheslavkol marked this pull request as ready for review July 10, 2026 10:50
@psmagin psmagin self-requested a review July 10, 2026 11:29
@viacheslavkol viacheslavkol requested a review from a team July 10, 2026 11:41
@viacheslavkol viacheslavkol merged commit f101c24 into master Jul 10, 2026
16 checks passed
@viacheslavkol viacheslavkol deleted the MODEXPW-630 branch July 10, 2026 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants