Skip to content
This repository has been archived by the owner on May 28, 2024. It is now read-only.

[HOLD] Update sw_format_ssim to stanford_mods_record.format_main #1005

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

aaron-collier
Copy link
Contributor

@aaron-collier aaron-collier commented Sep 25, 2023

Why was this change made? 🤔

Part of #992 - updates the sw_format_ssim facet.

Depends on: sul-dlss/cocina-models#626

How was this change tested? 🤨

⚡ ⚠ If this change has cross service impact, run integration tests that exercise indexing (e.g. searches in Argo for newly created/updated items, access_indexing_spec) and/or test in [stage|qa] environment, in addition to specs. ⚡

Copy link
Member

@mjgiarlo mjgiarlo left a comment

Choose a reason for hiding this comment

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

Should we mark this as HOLD until the cocina-models changes this needs are available, and the build passes?

@aaron-collier aaron-collier changed the title Update sw_format_ssim to stanford_mods_record.format_main [HOLD] Update sw_format_ssim to stanford_mods_record.format_main Sep 28, 2023
@ndushay
Copy link
Contributor

ndushay commented Oct 3, 2023

Specs make it look like "it's opposite day!" -- this seems to show significant logic differences between straight cocina indexing and stanford-mods indexing.

@ndushay
Copy link
Contributor

ndushay commented Oct 3, 2023

ACTION ITEM per discussion with @andrewjbtw @ndushay and @aaron-collier on 10/3/23: Aaron will write "report" tickets for sussing out the differences between stanford mods indexing and dor-indexing-app for this part of descriptive metadata.

@ndushay ndushay changed the title [HOLD] Update sw_format_ssim to stanford_mods_record.format_main [HOLD - doing report first] Update sw_format_ssim to stanford_mods_record.format_main Oct 5, 2023
@ndushay ndushay changed the title [HOLD - doing report first] Update sw_format_ssim to stanford_mods_record.format_main [HOLD] Update sw_format_ssim to stanford_mods_record.format_main Oct 5, 2023
@ndushay
Copy link
Contributor

ndushay commented Oct 9, 2023

waiting for cocina-models release with a patch, then reverting back to what it was doing and skipping this???

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants