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.

Sign up for GitHub

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

Jump to bottom

効果音はクライアント側で読み込むように変更 #85

Merged

sizuhiko merged 1 commit into main from pong-buffer-load-by-client

Mar 17, 2022

Contributor

sizuhiko commented Mar 17, 2022

変更の意図

サーバーで読み込んで毎回メッセージ送信すると、参加人数が多いとき負荷になる
以前はフロントのデプロイ先でなく、別のドメインにあった効果音は、音量調整してホストしたのでクロスドメインでなくなったため、フロントエンドで読み込み可能になった

変更概要

効果音のメッセージからバッファーを削除（メッセージ引数を変更）
クライアントは接続時に、効果音一覧の音声をすべてロードする
スピーカー画面は現時点どこからもリンクされないので、修正対象外（いずれ削除する


          pong buffer load by client

sizuhiko requested review from tastyHama and minakamoto

March 17, 2022 09:36

sizuhiko self-assigned this

Contributor Author

sizuhiko commented Mar 17, 2022

こちらデプロイして動作確認も実施するので、後日何か気になることあればコメントください

sizuhiko merged commit 6a6f9f4 into main

sizuhiko deleted the pong-buffer-load-by-client branch

March 17, 2022 09:38

sizuhiko mentioned this pull request

インターフェース仕様書をwikiに記載する #67

Closed

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