Skip to content

Conversation

@SKarolFolio
Copy link
Contributor

@SKarolFolio SKarolFolio commented Jan 9, 2026

Add Hubs search using LoC source and with local-availability support, and separate Hubs configuration for Search page and complex lookups.

https://folio-org.atlassian.net/browse/UILD-688

Technical details:

  • Added a Hub local-availability service with unit tests to detect and mark locally available resources.
  • Implemented a Hubs local-availability enricher to annotate search results (e.g., isLocal).
  • Introduced separate Search page and complex lookup configs and search strategies.
  • Refactored and extended formatters and table configs to support both Search and complex lookup contexts:
    • New Search page-specific result formatter and tests.
    • New UI formatters and components, including HubActionFormatter, HubSourceFormatter, and HubsLookupResultList, with unit tests.
    • Hook useHubsTableFormatter and updates to useFormattedResults to support lookup formatting.
  • Updated existing HubsResultList and related components to simplify formatting and loading handling and to use the new source/local labels.

…Config width, and update HubSourceFormatter for localization
…d tests; modify searchUIRegistry tests to check for limit property instead of searchableIndices
@github-actions
Copy link

github-actions bot commented Jan 9, 2026

Jest Unit Test Results

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit c0e83ae. ± Comparison against base commit e0b9e1a.

♻️ This comment has been updated with latest results.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 9, 2026


const enrichedData = hubData.map(hub => ({
...hub,
isLocal: localHubIds.has(hub.token),
Copy link
Contributor

@pkjacob pkjacob Jan 12, 2026

Choose a reason for hiding this comment

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

For this ticket, it is enough to have the isLocal boolean field in this object. However, for the next tickets, we will need the localHubId in this object.
For instance, if localHubId is truthy, then open the Edit screen for that hubId. Else, show the preview screen for the Hub to be imported from id.loc.gov.

@pkjacob pkjacob merged commit 9c0160d into master Jan 12, 2026
15 checks passed
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