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

Allow friends to join private battle directly #665

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jere0500
Copy link
Contributor

@jere0500 jere0500 commented May 3, 2024

PR for issue #654

Added a new checkbox in the hostBattleWindow which make it optionally possible to allow friends to join without a password. This checkbox sends a request in the battle lobby to set the gatekeeper to friends.

In PR: beyond-all-reason/teiserver#285
I would like to change the state of the gatekeeper friends to allow friends to join a private battle without entering a password.

Also when trying the to join a private lobby. The user directly tries to connect without a password.
(Users would directly join on reconnect, and as friends to private battles without having to click the join button)

Added checkbox, Setting the gatekeeper to friends.
Try to directly join, on lobby with a password (could be open for
friends)
@StanczakDominik
Copy link
Contributor

StanczakDominik commented Jun 11, 2024

The blocking PR has now been merged and deployed in Teiserver :)

@jere0500
Copy link
Contributor Author

Nice, thank you :)

@jere0500 jere0500 marked this pull request as ready for review June 11, 2024 12:24
@AntlerForce
Copy link
Collaborator

AntlerForce commented Sep 13, 2024

Testing the branch and this isn't working for me, after checking "Allow friends to join directly" and clicking Host it says a battle is spinning up but never arrives. The following error is in the log:

[t=00:01:23.326132][f=-000001] No cluster managers for, US
[t=00:01:23.523777][f=-000001] [liblobby] Error: Failed to match command: , SAYPRIVATE, , args:  !privatehost with pattern: , (%S+)%s+(.*)

@AntlerForce AntlerForce marked this pull request as draft September 13, 2024 20:17
@AntlerForce
Copy link
Collaborator

AntlerForce commented Sep 14, 2024

Got the host to spin up by bringing the changes over to my fork, but this feature isn't working how I'd expect. A password is still being provided but it doesn't work at all, so effectively no one except friends can join.
Screenshot_20240330

I would expect from the description that this gatekeeper feature would be like giving a fast-pass to friends, rather its just blocking everyone who isn't a friend, even if they have the correct password.

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.

3 participants