Skip to content
This repository was archived by the owner on Feb 27, 2023. It is now read-only.

pr feedback#465

Merged
kuruk-mm merged 1 commit into
refactor/new-voice-chat2from
tmp-menduz-1
Aug 31, 2022
Merged

pr feedback#465
kuruk-mm merged 1 commit into
refactor/new-voice-chat2from
tmp-menduz-1

Conversation

@menduz
Copy link
Copy Markdown
Member

@menduz menduz commented Aug 31, 2022

What?

...

Why?

...

@menduz menduz requested a review from a team as a code owner August 31, 2022 21:15
export function processVoiceFragment(message: Package<VoiceFragment>) {
const state = store.getState()
const voiceHandler = getVoiceHandler(state)
const profile = getCurrentUserProfile(state)
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Is this necessary?

Copy link
Copy Markdown
Member

@kuruk-mm kuruk-mm Aug 31, 2022

Choose a reason for hiding this comment

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

This code is from opus voice chat. I just moved it from the sagas to this new file. I tried to modify it least as possible.

@github-actions
Copy link
Copy Markdown
Contributor

Test this pull request

@kuruk-mm kuruk-mm merged commit e103aba into refactor/new-voice-chat2 Aug 31, 2022
@kuruk-mm kuruk-mm deleted the tmp-menduz-1 branch August 31, 2022 23:39
kuruk-mm added a commit that referenced this pull request Sep 8, 2022
* w

* checkpoint

* wip

* wip

* split voice chat sagas

* wip

* wip

* checkpoint: new voice sagas working with opus

* delete old voice sagas

* add missing libopus.wasm.js
add livekit voice handler

* fix

* change token handle

* update unity-renderer

* livekit with custom and temporal server to get token

* improve livekit voice chat

* room names by realm and island

* todo comment

* better logs

* fix: remove initialization of voice chat sagas from kernel (#453)

* feat: add spatial sound (need more test)
feat: add ff for livekit connection (independent from transport)

* checkpoint

* working for chrome

* remove logs

* typo

* fix ff

* fix

* fix

* remove undefined type

* use ff

* minor issues (#465)

Co-authored-by: menduz <github@menduz.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants