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

デバイス無効化時の Track の停止漏れを修正する #465

Merged
merged 8 commits into from
Oct 25, 2023

Conversation

tnamao
Copy link
Contributor

@tnamao tnamao commented Oct 23, 2023

変更履歴

  • [FIX] request media connect を実行した後に、Enable mic device Enable video device のトグルでデバイスを無効化した時に Media Processor が保持している Track の停止漏れを修正する
    • トグル切替でマイクやカメラのデバイスを無効化してもカメラ等のデバイスが使用中の状態のままになってしまっていた
    • @tnamao

@tnamao tnamao requested review from voluntas and sile October 23, 2023 08:36
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Oct 23, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 05c847e
Status: ✅  Deploy successful!
Preview URL: https://1248b338.sora-devtools.pages.dev
Branch Preview URL: https://feature-stop-media-processor.sora-devtools.pages.dev

View logs

@@ -1697,15 +1643,12 @@ export const setMicDevice = (micDevice: boolean) => {
}
} else if (state.soraContents.sora && state.soraContents.localMediaStream) {
// Sora 接続中の場合
stopLocalAudioTrack(dispatch, state);
Copy link
Member

Choose a reason for hiding this comment

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

👍

@voluntas
Copy link
Member

動作確認済みです。

@tnamao
Copy link
Contributor Author

tnamao commented Oct 24, 2023

細々とブラウザ依存対応などを入れてしまったので、お手数ですが動作確認してもらえると助かります 🙏

@voluntas
Copy link
Member

Chrome / Safari で動作確認済み。Safari はちょっとよくわからない挙動がありましたが、後回しで。

@voluntas
Copy link
Member

マージします。

@voluntas voluntas merged commit 8d8d687 into develop Oct 25, 2023
4 checks passed
@voluntas voluntas deleted the feature/stop-media-processor-on-device-disabled branch October 25, 2023 05:31
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