Skip to content

UIEH-1455 eHoldings Package record titlelist: Add Results list column chooser to Search within Actions button menu#1808

Merged
BogdanDenis merged 9 commits intomasterfrom
UIEH-1455
Apr 3, 2026
Merged

UIEH-1455 eHoldings Package record titlelist: Add Results list column chooser to Search within Actions button menu#1808
BogdanDenis merged 9 commits intomasterfrom
UIEH-1455

Conversation

@BogdanDenis
Copy link
Copy Markdown
Contributor

@BogdanDenis BogdanDenis commented Mar 27, 2026

Description

Show a column manager for Titles list in Package Show page.

Approach

To sync the column manager checkboxes and the MCL component - we need to use a single instance of useColumnManager hook. In our case the accordion header for Titles list and the list itself are separate components, so we need to call this hook in their common parent, which is <DetailsView>.
To use the hook, <DetailsView> was refactored to be a functional component. It's searchModal prop was renamed to renderAccordionHeaderSearch to better represent what it is on the UI. Some other components also use it, so had to update them to use the new prop name.

Also, updated several Jest tests, as always

Screenshots

chrome_dR2JdcFwT8.mp4

Issues

UIEH-1455

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 27, 2026

Jest Unit Test Results

    1 files  ±0    195 suites  ±0   4m 39s ⏱️ +33s
1 117 tests +2  1 114 ✅ +2  3 💤 ±0  0 ❌ ±0 
1 125 runs  +2  1 122 ✅ +2  3 💤 ±0  0 ❌ ±0 

Results for commit ad5225c. ± Comparison against base commit ed68f9c.

♻️ This comment has been updated with latest results.

@BogdanDenis BogdanDenis requested a review from a team March 30, 2026 14:05
@BogdanDenis BogdanDenis marked this pull request as ready for review March 30, 2026 14:06
@BogdanDenis BogdanDenis changed the title UIEH-1455 WIP UIEH-1455 eHoldings Package record titlelist: Add Results list column chooser to Search within Actions button menu Mar 30, 2026
@mariia-aloshyna mariia-aloshyna requested a review from a team April 1, 2026 15:53
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 3, 2026

@BogdanDenis BogdanDenis merged commit 7a6309c into master Apr 3, 2026
16 checks passed
@BogdanDenis BogdanDenis deleted the UIEH-1455 branch April 3, 2026 12:21
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