Skip to content

UIREC-350 Check holdings abandonment on pieces receiving form#698

Merged
usavkov-epam merged 4 commits intomasterfrom
UIREC-350
Feb 17, 2026
Merged

UIREC-350 Check holdings abandonment on pieces receiving form#698
usavkov-epam merged 4 commits intomasterfrom
UIREC-350

Conversation

@usavkov-epam
Copy link
Contributor

@usavkov-epam usavkov-epam commented Feb 7, 2026

Purpose

https://folio-org.atlassian.net/browse/UIREC-350

Implement handling and validation for abandoned holdings when receiving pieces on the form. This ensures proper management of holdings during batch receiving and prevents data loss when holdings are swapped or unlinked.

Approach

  • Added logic to detect and display a confirmation modal when abandoned holdings are found during piece receiving.
  • The modal shows relevant holdings information and allows the user to confirm or cancel the action.
  • Unit tests cover modal rendering, content validation, and user actions using userEvent.
  • Batch receive scenarios (with and without holdings swap) are supported and demonstrated in screencasts.

Screenshots

Batch pieces receive (no swap)

Screen.Recording.2026-02-09.at.20.50.43.mov

Batch pieces receive (swap holdings)

Screen.Recording.2026-02-09.at.20.55.29.mov

@usavkov-epam usavkov-epam self-assigned this Feb 7, 2026
@github-actions
Copy link

github-actions bot commented Feb 7, 2026

Jest Unit Test Results

452 tests  +25   450 ✅ +25   4m 6s ⏱️ +7s
106 suites + 2     2 💤 ± 0 
  1 files   ± 0     0 ❌ ± 0 

Results for commit 30255da. ± Comparison against base commit 6c06e86.

♻️ This comment has been updated with latest results.

@usavkov-epam usavkov-epam marked this pull request as ready for review February 10, 2026 09:15
@sonarqubecloud
Copy link

@usavkov-epam usavkov-epam merged commit 4e2d6a1 into master Feb 17, 2026
15 checks passed
@usavkov-epam usavkov-epam deleted the UIREC-350 branch February 17, 2026 11:48
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.

3 participants