Skip to content

WIP Ap 835 add alma provider - #104

Draft
jason-raitz wants to merge 6 commits into
AP-814-pdf-creation-dagfrom
AP-835_add-alma-provider
Draft

jason-raitz wants to merge 6 commits into
AP-814-pdf-creation-dagfrom
AP-835_add-alma-provider

Conversation

@jason-raitz

@jason-raitz jason-raitz commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

This will add an Alma provider (https://github.com/BerkeleyLibrary/mokelumne-providers-alma) to mokelumne.

To start out, the Alma provider only allows an unauthenticated user to query alma with an mms_id and get a single <record> marcxml string returned.

Util methods have been added to parse the item's record for languages according to Berkeley's historical use of marc fields. (This will match the legacy perl script's parsing)

A marc_tesseract_map.json has been added along with a util for using it in the marc util.

TODO:

  • refactor in parse_marc_xml.py
  • cleanup and linting

 - also fixes a couple of typos/bugs
 - rough sketch of marc_utils (will probably rolled into other util
@jason-raitz jason-raitz self-assigned this Aug 27, 2026

@anarchivist anarchivist left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good start. should this PR be against AP-814-pdf-creation-bag if @steve-sullivan is still working on that branch?

Comment thread mokelumne/util/marc_utils.py Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we change the name of this module to mokelumne.util.marc and incorporate the methods mokelumne.util.parse_marc_xml into it?

Comment thread mokelumne/util/marc_utils.py Outdated
Comment thread mokelumne/util/marc_utils.py Outdated
Comment thread mokelumne/util/marc_utils.py Outdated
@jason-raitz
jason-raitz changed the base branch from main to AP-814-pdf-creation-dag September 1, 2026 15:22
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