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

4.2.2 bugfix 0803 #3295

Merged
merged 13 commits into from
Aug 3, 2023
Merged

4.2.2 bugfix 0803 #3295

merged 13 commits into from
Aug 3, 2023

Conversation

Cilla-luodan
Copy link
Collaborator

No description provided.

@@ -65,7 +65,8 @@ int JoinChannel(FString token, FString channelId, int64 uid, const FChannelMedia
<parml>
<plentry id="token">
<pt>token</pt>
<pd>在服务端生成的用于鉴权的动态密钥。详见<xref keyref="guide-token"/>。</pd>
<pd>在服务端生成的用于鉴权的动态密钥。详见<xref keyref="guide-token"/>。
<note type="note">如果你需要同时加入多个频道或在频道间频繁切换,声网推荐你使用通配 Token 以避免每加入一个新的频道都需向服务端申请一个新的 Token,详见 <xref keyref="wildcard-token"/>。</note></pd>
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

“以避免”那一句话不如直接把优点说出来更直接,这样改一下是不是更好一点:
……,声网推荐你使用通配 Token 来提升加入频道或在频道间切换的速度,同时减小你的 Token 服务器的压力。详见 xxx

Copy link
Collaborator

Choose a reason for hiding this comment

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

我觉得这样写会有歧义,因为通配token本身并不能提升加快频道的速度,只是因为不需要每次加入一个新的频道就要重新生成token,所以省去了获取token的时间。

@@ -4528,7 +4528,7 @@
<keydef keys="CodecCapInfo" href="../API/class_codeccapinfo.dita">
<topicmeta>
<keywords>
<keyword>CodecCapAgoraVideoCodecCapInfoInfo</keyword>
<keyword>AgoraVideoCodecCapInfoInfo</keyword>
Copy link
Collaborator

Choose a reason for hiding this comment

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

InfoInfo -> Info

@@ -4243,7 +4243,7 @@
<keydef keys="CodecCapInfo" href="../API/class_codeccapinfo.dita">
<topicmeta>
<keywords>
<keyword>CodecCapAgoraVideoCodecCapInfoInfo</keyword>
<keyword>AgoraVideoCodecCapInfoInfo</keyword>
Copy link
Collaborator

Choose a reason for hiding this comment

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

InfoInfo -> Info

@@ -4528,7 +4528,7 @@
<keydef keys="CodecCapInfo" href="../API/class_codeccapinfo.dita">
<topicmeta>
<keywords>
<keyword>CodecCapAgoraVideoCodecCapInfoInfo</keyword>
<keyword>AgoraVideoCodecCapInfoInfo</keyword>
Copy link
Collaborator

Choose a reason for hiding this comment

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

InfoInfo -> Info

@@ -4243,7 +4243,7 @@
<keydef keys="CodecCapInfo" href="../API/class_codeccapinfo.dita">
<topicmeta>
<keywords>
<keyword>CodecCapAgoraVideoCodecCapInfoInfo</keyword>
<keyword>AgoraVideoCodecCapInfoInfo</keyword>
Copy link
Collaborator

Choose a reason for hiding this comment

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

InfoInfo -> Info

@@ -1062,7 +1063,6 @@
<li><xref keyref="AUDIO_PROCESSING_CHANNELS"/></li>
<li><xref keyref="AUDIO_PROFILE_TYPE"/></li>
<li><xref keyref="AUDIO_RECORDING_QUALITY_TYPE"/></li>
<li><xref keyref="AUDIO_REVERB_PRESET"/></li>
Copy link
Collaborator Author

@Cilla-luodan Cilla-luodan Aug 3, 2023

Choose a reason for hiding this comment

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

哈?为啥 cpp 里也删了?这个枚举类在 setLocalVoiceReverbPreset 里有用到,cpp apple unreal unity里都还在用的

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

VOICE_CHANGER_PRESET 那个也是,应该是只要处理 rn、elec、flutter 即可。

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@jinyuagora 记得处理一下 flutter 的 keysmap,有没删干净的

Copy link
Collaborator

Choose a reason for hiding this comment

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

哈?为啥 cpp 里也删了?这个枚举类在 setLocalVoiceReverbPreset 里有用到,cpp apple unreal unity里都还在用的

setLocalVoiceReverbPreset 和 setlocalvoicechanger 已无任何平台使用

Copy link
Collaborator

Choose a reason for hiding this comment

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

image image

@Nero-Hu
Copy link
Collaborator

Nero-Hu commented Aug 3, 2023

1

@Cilla-luodan Cilla-luodan merged commit 243cbfc into master Aug 3, 2023
31 checks passed
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.

4 participants