-
-
Notifications
You must be signed in to change notification settings - Fork 681
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 now show up as SSD on examine (+ soft ping port) #9705
Conversation
Why would we want to let players adjust how long until they are registered as being AFK? This should be a config setting and set entirely by the server. |
Ah, fair. I'm not sure why I did that. Should it be a separate config option or just the |
oh yea, inactivity period is probably fine |
Aight, I changed it to just use the default |
This should also show on MedHUDs |
@itsmeow like as in the "dead, distant soul" medhud? |
the yellow question mark |
@itsmeow done |
You have to actually call medhud update status when the afk value changes or it won't actually show. |
@itsmeow What'd be the best way to handle that? Since |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
About The Pull Request
This makes it so that AFK players will appear as SSD. This uses the inactivity period configuration, which is 5 minutes by default.
Although perhaps a different message could be shown for AFK players as opposed to SSD players? I'm open to suggestions, I'm not that good at coming up with flavor text.
Also ports tgstation/tgstation#66299, which fixes inactivity/
is_afk
not working, as uh, that kinda needs to work for this.Why It's Good For The Game
Makes it less likely for me to be killed when I'm AFK (but not disconnected) because I had to go use the bathroom or do some chores, due to someone not being able to tell I'm AFK when examining me.
Testing Photographs and Procedure
Screenshots&Videos
Soft-ping
AFK examine (+ working
is_afk
)Changelog
🆑 Absolucy, stylemistake
add: AFK players will now have flavor text indicating that they're AFK when examined.
fix: AFK detection now works properly again.
/:cl: