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 support for command !exit to show last user to exit the server #43

Closed
joerich opened this issue Jan 9, 2017 · 14 comments

Comments

Projects
3 participants
@joerich
Copy link

commented Jan 9, 2017

this would be useful to see who last exited the server

@alexanderkress

This comment has been minimized.

Copy link
Member

commented Jan 9, 2017

You mean a list of players, who left the server during a running map?
E.g. !exit -> "Last players who left: PlyA, PlyB, PlyC"
And when a new map starts this map will be reset and filled when players leave during an active map?
In case the nobody left and you use !exit, the output shall be "No players left during this match" ??

@alexanderkress alexanderkress self-assigned this Jan 9, 2017

@joerich

This comment has been minimized.

Copy link
Author

commented Jan 11, 2017

that would be ok, I was mainly thinking the last person leaving the server! But the last 3 would be ok too!

@alexanderkress

This comment has been minimized.

Copy link
Member

commented Jan 11, 2017

Hmm only the last leaving person is so much easier to implement... Now I was fighting already how to get a dynamic list of only the last three... If I would know from the beginning, that only one person would be enough, I could already have finished this feature :-)

Since this is your idea: only the last or the last 3? Your decision...

@joerich

This comment has been minimized.

Copy link
Author

commented Jan 11, 2017

Just the last would be fine!

and Thanks!!

@alexanderkress

This comment has been minimized.

Copy link
Member

commented Jan 11, 2017

Arf now when the code is almost finished :-)
Ok, only the last.
What would be the proper english:

  • option A: "The last player left: XX"
  • option B: "Last player left: XX"
  • option C: "Player last exited: XX"
  • option D: ??

And in case no player left: "No player left this match" ??

@joerich

This comment has been minimized.

Copy link
Author

commented Jan 11, 2017

it doesn't really matter! I guess option b?

@alexanderkress

This comment has been minimized.

Copy link
Member

commented Jan 11, 2017

ok option B will be used

@BidyBiddle

This comment has been minimized.

Copy link
Contributor

commented Jan 11, 2017

"Last disconnected player:" / "Last disconnect:" / "Last disconnection:" might be more appropriate.

Option A and B don't feel so smooth in english.

@alexanderkress

This comment has been minimized.

Copy link
Member

commented Jan 12, 2017

hmm yes you are right. but is "last disconnect:" correct regarding grammar??

"Last disconnected player: Fukushukumquat"
and UrT is already forcing a line-break :-)

@alexanderkress

This comment has been minimized.

Copy link
Member

commented Jan 12, 2017

and the name of the command? still "!exit" ? or "!disconnect"
if the command is "!exit", somehow the word "exit" should be in the output message or??

Damn how did it SG with their banexit command??

@joerich

This comment has been minimized.

Copy link
Author

commented Jan 12, 2017

I think maybe !exit its fairly easy to remember ;)

@alexanderkress

This comment has been minimized.

Copy link
Member

commented Feb 4, 2017

command will be in release 1.9, which will be release soon. want to test support to UrT 4.3.2 first...

@alexanderkress alexanderkress moved this from Todo to In Progress in Release 1.9.0 Feb 4, 2017

@alexanderkress alexanderkress moved this from In Progress to Done in Release 1.9.0 Feb 4, 2017

@joerich

This comment has been minimized.

Copy link
Author

commented Mar 1, 2017

I'd love to test it!

@alexanderkress

This comment has been minimized.

Copy link
Member

commented Mar 5, 2017

Feature is now in the develop branch available.
Happy testing :-)

Reconnected players are automatically removed from the "last disconnected player list".

@alexanderkress alexanderkress added this to the Release 1.9 milestone Mar 15, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.