Skip to content
This repository has been archived by the owner on Feb 5, 2025. It is now read-only.

improve voice logic #483

Merged

Conversation

z4o4z
Copy link
Member

@z4o4z z4o4z commented Dec 30, 2024

Fixes or implements VF-XXX

Brief description. What is this change?

Implementation details. How do you make this change?

Setup information

Deployment Notes

Related PRs

Checklist

  • Breaking changes have been communicated, including:
    • New required environment variables
    • Renaming of interfaces (API routes, request/response interface, etc)
  • New environment variables have been deployed
  • Appropriate tests have been written
    • Bug fixes are accompanied by an updated or new test
    • New features are accompanied by a new test

Copy link

linear bot commented Dec 30, 2024

Copy link

vercel bot commented Dec 30, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
react-chat ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 6, 2025 5:34pm

Copy link

graphite-app bot commented Dec 30, 2024

How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • merge - adds this PR to the back of the merge queue
  • hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has required the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

@z4o4z z4o4z changed the base branch from master to andrew/voice-widget-updates/DX-000 December 30, 2024 19:48
@z4o4z z4o4z force-pushed the evgeny/improve-voice-logic/COR-4354 branch from 29f3853 to bb8a041 Compare December 30, 2024 19:50
Copy link
Member Author

z4o4z commented Dec 30, 2024

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more


How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • merge - adds this PR to the back of the merge queue
  • hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has required the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

sourceOpen = true;
ref.outputSourceBuffer = outputMediaSource.addSourceBuffer('audio/mpeg');

ref.outputSourceBuffer.abort
Copy link

Choose a reason for hiding this comment

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

Looks like there's a missing semicolon and parentheses on this line. The correct syntax should be ref.outputSourceBuffer.abort();

Spotted by Graphite Reviewer

Is this helpful? React 👍 or 👎 to let us know.

Copy link
Member Author

Choose a reason for hiding this comment

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

this is a legacy implementation, can be removed

@z4o4z z4o4z merged commit fb57b93 into andrew/voice-widget-updates/DX-000 Jan 7, 2025
8 of 11 checks passed
@z4o4z z4o4z deleted the evgeny/improve-voice-logic/COR-4354 branch January 7, 2025 11:44
sofly pushed a commit that referenced this pull request Jan 7, 2025
sofly pushed a commit that referenced this pull request Jan 7, 2025
sofly pushed a commit that referenced this pull request Jan 7, 2025
sofly pushed a commit that referenced this pull request Jan 7, 2025
sofly pushed a commit that referenced this pull request Jan 7, 2025
sofly pushed a commit that referenced this pull request Jan 7, 2025
sofly pushed a commit that referenced this pull request Jan 7, 2025
sofly pushed a commit that referenced this pull request Jan 7, 2025
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.

1 participant