Skip to content

MODSOURMAN-1371 Remove quick-marc events handling logic#1020

Merged
SvitlanaKovalova1 merged 4 commits intomasterfrom
feature/qm-unify
Mar 25, 2026
Merged

MODSOURMAN-1371 Remove quick-marc events handling logic#1020
SvitlanaKovalova1 merged 4 commits intomasterfrom
feature/qm-unify

Conversation

@SvitlanaKovalova1
Copy link
Copy Markdown
Contributor

@SvitlanaKovalova1 SvitlanaKovalova1 commented Mar 23, 2026

Purpose

MODSOURMAN-1371 Clean-up unused logic after unifying MARC Create/Update Flows in mod-quick-marc

Approach

Remove quick-marc events handling logic

Karate tests:

image

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.

@SvitlanaKovalova1 SvitlanaKovalova1 marked this pull request as ready for review March 24, 2026 10:44
Copy link
Copy Markdown
Contributor

@psmagin psmagin left a comment

Choose a reason for hiding this comment

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

I think major module version should be bumped

Comment thread mod-source-record-manager-client/pom.xml Outdated
@sonarqubecloud
Copy link
Copy Markdown

@SvitlanaKovalova1 SvitlanaKovalova1 merged commit eca06a2 into master Mar 25, 2026
6 checks passed
@SvitlanaKovalova1 SvitlanaKovalova1 deleted the feature/qm-unify branch March 25, 2026 16:31
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