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

AFK players will no longer default to continue playing during restart votes. #20323

Merged
merged 1 commit into from Sep 6, 2016

Conversation

MrStonedOne
Copy link
Member

@MrStonedOne MrStonedOne commented Sep 4, 2016

🆑
tweak: AFK players will no longer default to continue playing during restart votes.
/:cl:

Non afk players will still default if they don't vote.

…anged to the default.

ie: no afks counting as continue playing during restart votes.
@AnturK AnturK added the Tweak label Sep 4, 2016
@WJohn
Copy link
Contributor

WJohn commented Sep 4, 2016

Well. This should make restarting rounds about 1000x more likely.

@PKPenguin321
Copy link
Contributor

👍 Thank you. A lot of the time if most players die and it's a lagfest, many of them alt-tab, and miss restart votes

@harmonyn
Copy link

harmonyn commented Sep 4, 2016

it still requires admin confirmation right?

@PKPenguin321
Copy link
Contributor

Speaking of admin confirmation, it shouldn't be needed if the online admins are AFK

@silicons
Copy link
Contributor

silicons commented Sep 4, 2016

i ded pls restart

@MrStonedOne
Copy link
Member Author

Speaking of admin confirmation, it shouldn't be needed if the online admins are AFK

Do you think you are playing with children? When I coded the admin check i gave it both a check for afk, AND a check for rather or not the admin online had the power to restart the server manually themselves.

@Kromgar
Copy link
Contributor

Kromgar commented Sep 5, 2016

had rounds where admins refused to restart and it was laggy to all hell

@PKPenguin321
Copy link
Contributor

oh shit mso, one step ahead of me

good stuff all around 👍

@Bawhoppen
Copy link
Contributor

BEST PR OF 2016 👍x💯x💯x💯x💯x💯x💯x💯x💯x💯x💯x💯x💯x💯x💯x💯x💯x💯x💯x💯x💯x💯

@phil235 phil235 merged commit 9005d8d into tgstation:master Sep 6, 2016
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.

None yet

9 participants