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

Version 2.5.0 #257

Merged
merged 9 commits into from
Dec 7, 2023
Merged

Version 2.5.0 #257

merged 9 commits into from
Dec 7, 2023

Conversation

nmanu1
Copy link
Contributor

@nmanu1 nmanu1 commented Dec 7, 2023

cea2aj and others added 9 commits June 29, 2023 13:26
Merge master (v2.4.0) into develop
Introduce document vertical support

Add segment, document, and documents to the result object

J=BACK-2523
TEST=manual

I connected the local test site to an experience with a document vertical setup and I confirmed that the segment was shown on the result. I still need to test the document and documents fields
The type "Document" was conflicting with the same type in node and in the browser which was causing some TS issues. Renaming this fixes this conflict.

J=BACK-2523
TEST=compile
It turns out this change was unnecessary because someone could pass an array as the generic

J=BACK-2523
TEST=compile
Merge master (v2.4.1) into develop
Bump package version to v2.5.0.

J=none
TEST=auto
@nmanu1 nmanu1 marked this pull request as ready for review December 7, 2023 21:27
@nmanu1 nmanu1 requested a review from a team as a code owner December 7, 2023 21:27
@coveralls
Copy link

Pull Request Test Coverage Report for Build 7134221283

  • 0 of 3 (0.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-1.3%) to 89.352%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/transformers/searchservice/ResultsFactory.ts 0 3 0.0%
Totals Coverage Status
Change from base Build 7104366154: -1.3%
Covered Lines: 219
Relevant Lines: 239

💛 - Coveralls

@nmanu1
Copy link
Contributor Author

nmanu1 commented Dec 7, 2023

Will make an item to address the decrease in test coverage!

@nmanu1 nmanu1 merged commit 606c089 into master Dec 7, 2023
15 checks passed
nmanu1 added a commit that referenced this pull request Dec 7, 2023
- Added support for document verticals (#249, #251, #252)
nmanu1 added a commit that referenced this pull request Dec 7, 2023
Changes
- Added support for document verticals (#249, #251, #252)
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.

5 participants