Skip to content
This repository was archived by the owner on Nov 29, 2023. It is now read-only.

Conversation

nashley
Copy link
Contributor

@nashley nashley commented Apr 7, 2023

⚠️Please do not make any changes to spec files in this repository!⚠️

All spec changes should be done in the api-specs repository.

Please follow the Guide for contributing to our developer documentation.

Spec PR here

Has a lot of loose ends to tie up. For example:

  • Properly adding StartTranscription to about, overview, etc
  • Code examples for StartTranscription
  • Properly adding StopTranscription to about, overview, etc
  • Code examples for StopTranscription
  • Add WebSocket messages to overview
  • Double check everything for consistency
  • Update spec in separate repo
  • Add startTranscription guide
  • Use real examples after implementing before merging
  • Rename realtimeTranscription to startTranscription?

@github-actions
Copy link
Contributor

github-actions bot commented Apr 7, 2023

@github-actions
Copy link
Contributor

github-actions bot commented Apr 7, 2023

@github-actions
Copy link
Contributor

github-actions bot commented Apr 7, 2023

@github-actions
Copy link
Contributor

github-actions bot commented Apr 7, 2023

@github-actions
Copy link
Contributor

github-actions bot commented Apr 7, 2023

1 similar comment
@github-actions
Copy link
Contributor

@nashley nashley force-pushed the vapi-1390-real-time-transcription branch from 72f46d3 to 7bcfda2 Compare April 19, 2023 14:10
@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

github-actions bot commented Jun 8, 2023

| callUrl | The URL of the call associated with the event |
| cause | The reason the real-time transcription was rejected, can be one of `connection-error`, `duplicate`, `error` or `track-limit` |
| direction | The direction of the call. Either `inbound` or `outbound`. The direction of a call never changes. |
| enqueuedTime | (optional) If [call queueing](/apis/voice#operation/createCall/) is enabled and this is an outbound call, time the call was queued, in ISO 8601 format. |
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
| enqueuedTime | (optional) If [call queueing](/apis/voice#operation/createCall/) is enabled and this is an outbound call, time the call was queued, in ISO 8601 format. |
| enqueuedTime | (optional) If [call queueing](/apis/voice#operation/createCall/) is enabled and this is an outbound call, time the call was queued, in ISO 8601 format. Else, this field is omitted. |

Copy link
Contributor

Choose a reason for hiding this comment

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

Got this in all other callbacks, thx!

@@ -4197,9 +4197,7 @@ paths:
description: >-
Copy link
Contributor

Choose a reason for hiding this comment

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

I can't comment on the line above this, but maybe we should rename the summary to Get Recording Transcription?

Co-authored-by: Catherine <31828886+CatexB@users.noreply.github.com>
@github-actions
Copy link
Contributor

github-actions bot commented Jun 9, 2023

…thub.com:Bandwidth/api-docs into vapi-1390-real-time-transcription
@github-actions
Copy link
Contributor

github-actions bot commented Jun 9, 2023

@github-actions
Copy link
Contributor

@marcelohossomi marcelohossomi marked this pull request as ready for review June 29, 2023 14:26
@marcelohossomi marcelohossomi requested a review from a team as a code owner June 29, 2023 14:26
@marcelohossomi marcelohossomi requested a review from a team June 29, 2023 14:26
@github-actions
Copy link
Contributor

tmoney-bw
tmoney-bw previously approved these changes Jun 29, 2023
title: Start Transcription
slug: /voice/bxml/startTranscription
description: A general overview of Bandwidth's startTranscription BXML Verb
keywords:
Copy link
Contributor

Choose a reason for hiding this comment

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

small but transcription should be here probably

Copy link
Contributor

Choose a reason for hiding this comment

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

same for the other places ig

Copy link
Contributor

Choose a reason for hiding this comment

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

It has "transcribing"

This event may be sent to the url specified when sending a [`<StartTranscription>`][1] verb. This event will be sent if the real-time transcription requested in the `<StartTranscription>` verb was not started.

## Request Parameters
TODO `s/originalTranscription/originalRealtimeTranscription`?
Copy link
Contributor

Choose a reason for hiding this comment

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

TODO

Copy link
Contributor

Choose a reason for hiding this comment

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

Removed

@github-actions
Copy link
Contributor

@marcelohossomi marcelohossomi merged commit bda3a2a into main Jun 30, 2023
@marcelohossomi marcelohossomi deleted the vapi-1390-real-time-transcription branch June 30, 2023 03:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants