Skip to content

DRYD-2144 Authority Fields > Predictive Search Result Messaging - #28

Merged
spirosdi merged 1 commit into
mainfrom
feature/dryd-2144
Jul 27, 2026
Merged

DRYD-2144 Authority Fields > Predictive Search Result Messaging#28
spirosdi merged 1 commit into
mainfrom
feature/dryd-2144

Conversation

@spirosdi

Copy link
Copy Markdown
Contributor

What does this do?
Adds a new formatNarrowResultsMessage prop to AutocompleteInput, and shows that message ("Continue typing to narrow results" by default) in the dropdown header whenever any of the field's sources returns a truncated result list. A source is considered truncated when the server reports more matches than it returned, or when it returned a full page of results. The search service caps totalItems at the page size (https://collectionspace.atlassian.net/browse/DRYD-2157), so a full page is the only reliable truncation signal. For multi-vocabulary sources, each vocabulary is checked independently. One truncated vocabulary is enough to show the message.

Why are we doing this? (with JIRA link)
https://collectionspace.atlassian.net/browse/DRYD-2144. The message that states: “40 matching terms found” is confusing because this actually translates to at least 40 and not exactly 40.

How should this be tested? Do these changes have associated tests?
Tests have been added. For manual testing please follow the steps: collectionspace/cspace-ui.js#367

Dependencies for merging? Releasing to production?
It should be released/published along the collectionspace/cspace-ui.js#367.

Has the application documentation been updated for these changes?
Changelog has been updated

Did someone actually run this code to verify it works?
@spirosdi ran it locally

…, displayed when any source returns a truncated result list;
@spirosdi
spirosdi merged commit 4750fd8 into main Jul 27, 2026
5 checks passed
@spirosdi
spirosdi deleted the feature/dryd-2144 branch July 27, 2026 04:57
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.

2 participants