MediaStreamRecorder icon indicating copy to clipboard operation
MediaStreamRecorder copied to clipboard

Feature Request: MultiStreamRecorder - removeStream

Open wootencl opened this issue 7 years ago • 2 comments

Would be nice if there was an additional helper method that allowed for the removal of a steam for the MultiStreamRecorder. First idea would be to potentially pass in the MediaStream object as ...removeStream(stream) then based on its id delete it from the arrayOfMediaStreams. Need to look at the source a little longer to see how this would effect the stream capture. If I have some free time in the future I can probably try to put a PR with the implementation if that would be of interest.

wootencl avatar Jul 19 '17 03:07 wootencl

Hello, any updates on this?

I'm currently developing a video conference-like app that can record the videos of users (max of 4) with MultiStreamRecorder. And when a user leaves the room, his placement from the video will become blank and would be available for other user that will join the meeting. Since there is an existing "addStream" method, it would be nice if there would also be a "removeStream" method.

Also, since there there is no method for saving the video, I added a this.save method from the save method of MediaStreamRecorder.

kim00kim avatar Apr 18 '18 08:04 kim00kim

Hi @kim00kim, I actually ended up forking and making the modifications for removal. Could probably make a PR to try and get it into the main repo but haven't had the time. Here's my fork with the related logic if that's of interest: https://github.com/wootencl/MediaStreamRecorder

wootencl avatar Apr 18 '18 14:04 wootencl