Skip to content

MODSOURMAN-1353: Remove relatedRecordVersion from quickMarc flow#990

Merged
psmagin merged 3 commits into
masterfrom
MODSOURMAN-1353
Sep 12, 2025
Merged

MODSOURMAN-1353: Remove relatedRecordVersion from quickMarc flow#990
psmagin merged 3 commits into
masterfrom
MODSOURMAN-1353

Conversation

@psmagin

@psmagin psmagin commented Sep 3, 2025

Copy link
Copy Markdown
Contributor

Purpose

  1. relatedRecordVersion is no longer expected in quickMarc flow.
  2. All logic that relies on relatedRecordVersion is removed.
  3. GET /change-manager/parsedRecords/{id}endpoint is deleted.
  4. PUT /change-manager/parsedRecords/{id} endpoint is updated

Is this change testable? If not - why?

Checklist

  • I have updated NEWS.md.
  • I have added javadocs to new methods.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation e.g. README.md.
  • I have ran karate tests against this feature.

@psmagin psmagin force-pushed the MODSOURMAN-1353 branch 3 times, most recently from 6e6246e to 13bac89 Compare September 10, 2025 08:01
@psmagin psmagin marked this pull request as ready for review September 11, 2025 12:47
@psmagin psmagin requested review from a team, SvitlanaKovalova1 and viacheslavkol and removed request for a team September 11, 2025 12:47
@psmagin psmagin self-assigned this Sep 11, 2025
@psmagin

psmagin commented Sep 11, 2025

Copy link
Copy Markdown
Contributor Author

The PR will also include changes from folio-org/data-import-raml-storage#321

@sonarqubecloud

Copy link
Copy Markdown

@psmagin psmagin merged commit 50a6456 into master Sep 12, 2025
8 checks passed
@psmagin psmagin deleted the MODSOURMAN-1353 branch September 12, 2025 14:46
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.

4 participants