Skip to content
This repository has been archived by the owner on May 24, 2018. It is now read-only.

Added login timeout #57

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Added login timeout #57

wants to merge 2 commits into from

Conversation

alejandroliu
Copy link

Implemented the login timer functionality.

@shoghicp
Copy link
Member

If a setting is an integer, Don't require another type to disable it. Instead use 0 or -1, as done on other confit settings

@alejandroliu
Copy link
Author

I was following what was written in config.yml

#Number of seconds to wait for a player to be authenticated. If false, disables the timeout

@alejandroliu
Copy link
Author

OK, it uses 0 or -1 to disable timeouts.

@PEMapModder
Copy link
Contributor

Type of false: Boolean
Type of 0 or -1: Integer
It would be good to use the same type (Integer)

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.

3 participants