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

Add a warning so others dont have to suffer like me. #10022

Open
TrevorBlythe opened this issue Sep 30, 2024 · 1 comment
Open

Add a warning so others dont have to suffer like me. #10022

TrevorBlythe opened this issue Sep 30, 2024 · 1 comment

Comments

@TrevorBlythe
Copy link

TrevorBlythe commented Sep 30, 2024

Your Godot version:
4.2.2

Issue description:
I got brutally DESTROYED by this. please dont let other suffers like me. Add this line in the docs:

Warning: WebRTCPeerConnection cannot accept  remote ICE candidates until a remote session description is set. This means, 
set the remote description before adding ice candidates.

URL to the documentation page (if already existing):
https://docs.godotengine.org/en/stable/tutorials/networking/webrtc.html
AND
https://docs.godotengine.org/en/stable/classes/class_webrtcpeerconnection.html#class-webrtcpeerconnection

put it in both plz..

Tasks

No tasks being tracked yet.
@TrevorBlythe
Copy link
Author

Trust me, if you dont think this is needed, let me tell you it is. I was stuck suffering for DAYS (suffering) because two lines of code were in the wrong ORDER? and it shoudn't matter tbh it should buffer ice candidates until its ready to accept but they shut that idea down REAL quick.

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

No branches or pull requests

1 participant