Skip to content

feat(video): Add post call transcription#609

Merged
dragonmantank merged 2 commits intomainfrom
devx-10218-add-transcription-to-video
Feb 6, 2026
Merged

feat(video): Add post call transcription#609
dragonmantank merged 2 commits intomainfrom
devx-10218-add-transcription-to-video

Conversation

@dragonmantank
Copy link
Member

@dragonmantank dragonmantank commented Feb 6, 2026

Added post call transcription options for Video

Contribution Checklist

@dragonmantank dragonmantank requested a review from Copilot February 6, 2026 20:10
@codecov-commenter
Copy link

codecov-commenter commented Feb 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.93%. Comparing base (64180b0) to head (f23d260).

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #609   +/-   ##
=========================================
  Coverage     98.93%   98.93%           
- Complexity     3607     3610    +3     
=========================================
  Files           542      542           
  Lines          7953     7987   +34     
  Branches        443      443           
=========================================
+ Hits           7868     7902   +34     
  Misses           64       64           
  Partials         21       21           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds post-call transcription support to the Video Archive model and expands test coverage with JSON fixtures and new unit tests around transcription request/response fields.

Changes:

  • Added transcription-related fields and nested types to Archive plus builder support for request options.
  • Added new unit tests and JSON fixtures covering serialization/deserialization of transcription fields.
  • Updated CHANGELOG.md to note the new Video transcription capability.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 3 comments.

File Description
src/main/java/com/vonage/client/video/Archive.java Adds transcription request/response fields, nested models, and builder methods.
src/test/java/com/vonage/client/video/ArchiveTest.java Adds tests and helper methods to validate transcription JSON serialization/deserialization.
src/test/resources/com/vonage/client/video/archive-*.json Adds JSON fixtures for new transcription scenarios.
CHANGELOG.md Documents the new post-call transcription options.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

All notable changes to this project will be documented in this file.
This project adheres to [Semantic Versioning](http://semver.org/).

# Unreleasd
Copy link

Copilot AI Feb 6, 2026

Choose a reason for hiding this comment

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

Typo in the changelog section header: 'Unreleasd' should be 'Unreleased' (and ideally match existing section header formatting).

Suggested change
# Unreleasd
# [Unreleased]

Copilot uses AI. Check for mistakes.
*
* @return This builder.
*/
public Builder hasTranscription(boolean hasTranscription) {
Copy link

Copilot AI Feb 6, 2026

Choose a reason for hiding this comment

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

Using a primitive boolean here makes it impossible to represent 'unset' once the method is called (and may cause hasTranscription: false to be serialized when callers didn’t intend to send the field). Consider changing the parameter to Boolean, or adding an overload like hasTranscription(Boolean hasTranscription) (or an enableTranscription() convenience method) so API consumers can explicitly control omission vs. false.

Suggested change
public Builder hasTranscription(boolean hasTranscription) {
public Builder hasTranscription(boolean hasTranscription) {
return hasTranscription(Boolean.valueOf(hasTranscription));
}
/**
* (OPTIONAL) Whether the archive will have transcription.
* Allows {@code null} to represent an unset value, so the field can be omitted.
*
* @param hasTranscription {@code Boolean.TRUE} to enable transcription,
* {@code Boolean.FALSE} to disable it,
* or {@code null} to leave it unset.
*
* @return This builder.
*/
public Builder hasTranscription(Boolean hasTranscription) {

Copilot uses AI. Check for mistakes.
@dragonmantank dragonmantank merged commit c5c0999 into main Feb 6, 2026
16 checks passed
@dragonmantank dragonmantank deleted the devx-10218-add-transcription-to-video branch February 6, 2026 20:30
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