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

feat(html5): Added captionLocale property and related logic #20272

Open
wants to merge 10 commits into
base: v3.0.x-release
Choose a base branch
from

Conversation

GuiLeme
Copy link
Collaborator

@GuiLeme GuiLeme commented May 15, 2024

What does this PR do?

It adds the captionLocale property to the user table and also changes related logic, so now it is possible for the user to "speak" and produce captions in one language and read captions in another one.

Motivation

So with the plugins, and the upcoming migration of the typed captions, this PR will not only enhance the usability for the plugin dev, but it will also give them the opportunity to develop a translation plugin in which instead of transcript what one is saying, they will translate it, even though their speechLocale is different from what they are receiving in the captions.

@ramonlsouza ramonlsouza added this to the Release 3.0 milestone May 15, 2024
@GuiLeme GuiLeme requested a review from Tainan404 May 15, 2024 13:31
Copy link

This pull request has conflicts ☹
Please resolve those so we can review the pull request.
Thanks.

@GuiLeme GuiLeme requested a review from gustavotrott May 23, 2024 12:17
Copy link

This pull request has conflicts ☹
Please resolve those so we can review the pull request.
Thanks.

Copy link

sonarcloud bot commented May 27, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

Copy link

Automated tests Summary

All the CI tests have passed!

Copy link
Collaborator

@gustavotrott gustavotrott left a comment

Choose a reason for hiding this comment

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

I reviewed only the backend code.
It looks good!

Copy link

This pull request has conflicts ☹
Please resolve those so we can review the pull request.
Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants