Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WEBDEV-5842 Add collectionTitles property to match new PPS responses #33

Merged
merged 2 commits into from
Feb 13, 2023

Conversation

latonv
Copy link
Contributor

@latonv latonv commented Feb 10, 2023

This PR adds a collectionTitles property to SearchResponseDetails objects, which optionally contains any collection titles object found on the response. The PPS has recently been upgraded to include such an object on hits/aggregations requests, containing a mapping from any collection identifiers to their readable titles.

@github-actions
Copy link

github-actions bot commented Feb 10, 2023

PR Preview Action v1.2.0
Preview removed because the pull request was closed.
2023-02-13 23:12 UTC

@codecov
Copy link

codecov bot commented Feb 10, 2023

Codecov Report

Merging #33 (bfa87d3) into main (5f54ef1) will increase coverage by 0.41%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #33      +/-   ##
==========================================
+ Coverage   86.73%   87.15%   +0.41%     
==========================================
  Files          26       26              
  Lines        2511     2522      +11     
  Branches      122      124       +2     
==========================================
+ Hits         2178     2198      +20     
+ Misses        321      313       -8     
+ Partials       12       11       -1     
Impacted Files Coverage Δ
src/responses/search-response-details.ts 94.59% <100.00%> (+9.59%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@latonv latonv merged commit dc60d7f into main Feb 13, 2023
@latonv latonv deleted the webdev5842-collection-titles branch February 13, 2023 23:10
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.

None yet

2 participants