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

DELIA-62843:TTS thunder plugin missing "speechrate" attribute #272

Merged
merged 2 commits into from
Aug 8, 2023
Merged

DELIA-62843:TTS thunder plugin missing "speechrate" attribute #272

merged 2 commits into from
Aug 8, 2023

Conversation

vdinak240
Copy link
Contributor

Reason for change: TTS thunder plugin missing "speechrate"
Test Procedure: Mentioned in ticket
Risks: Low

vdinak240 and others added 2 commits August 7, 2023 11:31
Reason for change: TTS thunder plugin missing "speechrate"
Test Procedure: Mentioned in ticket
Risks: Low

Signed-off-by: vdinak240 <[email protected]>
@pwielders pwielders self-requested a review August 8, 2023 09:25
Copy link
Collaborator

@pwielders pwielders left a comment

Choose a reason for hiding this comment

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

This is a breaking interface change and for Metro development not allowed (Metro within major releases is bound to be backwards compatible). Since this interface is only implemented by Comcast I assume you know what you are doing ;-) Do approved it with this warning...

@pwielders pwielders merged commit 5b153c3 into rdkcentral:master Aug 8, 2023
10 checks passed
cmurug728 pushed a commit to cmurug728/ThunderInterfaces that referenced this pull request Sep 14, 2023
…tral#272)

Reason for change: TTS thunder plugin missing "speechrate"
Test Procedure: Mentioned in ticket
Risks: Low

Signed-off-by: vdinak240 <[email protected]>
Co-authored-by: Pierre Wielders <[email protected]>
cmurug728 pushed a commit to cmurug728/ThunderInterfaces that referenced this pull request Sep 14, 2023
…tral#272)

Reason for change: TTS thunder plugin missing "speechrate"
Test Procedure: Mentioned in ticket
Risks: Low

Signed-off-by: vdinak240 <[email protected]>
Co-authored-by: Pierre Wielders <[email protected]>
karuna2git pushed a commit that referenced this pull request Sep 14, 2023
…290)

Reason for change: TTS thunder plugin missing "speechrate"
Test Procedure: Mentioned in ticket
Risks: Low

Signed-off-by: vdinak240 <[email protected]>
Co-authored-by: Vishnu Dinakaran <[email protected]>
Co-authored-by: Pierre Wielders <[email protected]>
pwielders added a commit that referenced this pull request Sep 21, 2023
…291)

Reason for change: TTS thunder plugin missing "speechrate"
Test Procedure: Mentioned in ticket
Risks: Low

Signed-off-by: vdinak240 <[email protected]>
Co-authored-by: Vishnu Dinakaran <[email protected]>
Co-authored-by: Pierre Wielders <[email protected]>
volkan-aslan pushed a commit that referenced this pull request Nov 1, 2023
Reason for change: TTS thunder plugin missing "speechrate"
Test Procedure: Mentioned in ticket
Risks: Low

Signed-off-by: vdinak240 <[email protected]>
Co-authored-by: Pierre Wielders <[email protected]>
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.

2 participants