Skip to content

Conversation

@SimonBrandner
Copy link
Contributor

@SimonBrandner SimonBrandner commented Jul 14, 2022

For element-hq/element-call#468


This change is marked as an internal change (Task), so will not be included in the changelog.

Signed-off-by: Šimon Brandner <[email protected]>
@SimonBrandner SimonBrandner added the T-Task Tasks for the team like planning label Jul 14, 2022
@SimonBrandner SimonBrandner marked this pull request as ready for review July 15, 2022 11:17
@SimonBrandner SimonBrandner requested a review from a team as a code owner July 15, 2022 11:17
@robintown
Copy link
Member

I'm trying to understand how this works - is localVolume or the event hooked up to anything?

@SimonBrandner
Copy link
Contributor Author

I'm trying to understand how this works - is localVolume or the event hooked up to anything?

Yes, it is in element-hq/element-call#468

@robintown
Copy link
Member

Oh, I see, volume has to be set on the media element in the front-end

@SimonBrandner
Copy link
Contributor Author

(going to block this until the element-call side is ready)

@SimonBrandner
Copy link
Contributor Author

Unblocking and merging as the element-call side got product approval

@SimonBrandner SimonBrandner merged commit e876482 into robertlong/group-call Jul 25, 2022
@SimonBrandner SimonBrandner deleted the SimonBrandner/feat/local-volume branch July 25, 2022 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-Task Tasks for the team like planning

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants