Skip to content

UIIN-3625: The page reloads instead of showing "Confirm move" modal when moving holdings to another instance in member tenant#3002

Closed
mariia-aloshyna wants to merge 3 commits intomasterfrom
UIIN-3627
Closed

UIIN-3625: The page reloads instead of showing "Confirm move" modal when moving holdings to another instance in member tenant#3002
mariia-aloshyna wants to merge 3 commits intomasterfrom
UIIN-3627

Conversation

@mariia-aloshyna
Copy link
Copy Markdown
Contributor

Purpose

Fix a first-click failure in ECS holdings move flow where clicking Move to could trigger a page reload/remount instead of opening the confirmation modal, especially for instances with multiple holdings and acquisition-linked data.

Approach

  • Added explicit button semantics and guarded pointer/click events to prevent unintended parent/default behavior.
  • Safely handles unresolved/invalid linked holdings before rendering modal content.
  • Normalizes holdings IDs used in move/linkage logic to avoid invalid entries.
  • Uses correct tenant source in modal holdings context.

Refs

https://folio-org.atlassian.net/browse/UIIN-3625

Screenshots

@mariia-aloshyna mariia-aloshyna changed the title UIIN-3627: The page reloads instead of showing "Confirm move" modal when moving holdings to another instance in member tenant UIIN-3625: The page reloads instead of showing "Confirm move" modal when moving holdings to another instance in member tenant Apr 9, 2026
@mariia-aloshyna mariia-aloshyna deleted the UIIN-3627 branch April 9, 2026 12:37
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 9, 2026

Quality Gate Failed Quality Gate failed

Failed conditions
48.5% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

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.

1 participant