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 now show up as SSD on examine (+ soft ping port) #9705

Merged
merged 13 commits into from
Oct 19, 2023

Conversation

Absolucy
Copy link
Contributor

@Absolucy Absolucy commented Aug 18, 2023

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

image

AFK examine (+ working is_afk)

image

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:

@Absolucy Absolucy requested a review from itsmeow as a code owner August 18, 2023 20:13
@github-actions github-actions bot added the TGUI-Changes Contains changes to TGUI. Make sure its up to date with TGUI 4.0 label Aug 18, 2023
@PowerfulBacon
Copy link
Member

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.

@Absolucy
Copy link
Contributor Author

Absolucy commented Aug 18, 2023

Ah, fair. I'm not sure why I did that.

Should it be a separate config option or just the INACTIVITY_PERIOD config?

@PowerfulBacon
Copy link
Member

oh yea, inactivity period is probably fine

@Absolucy
Copy link
Contributor Author

Aight, I changed it to just use the default is_afk arguments, which is the inactivity period configured by the server.

@itsmeow
Copy link
Member

itsmeow commented Aug 30, 2023

This should also show on MedHUDs

@Absolucy
Copy link
Contributor Author

Absolucy commented Sep 4, 2023

@itsmeow like as in the "dead, distant soul" medhud?

@itsmeow
Copy link
Member

itsmeow commented Sep 4, 2023

the yellow question mark

@Absolucy
Copy link
Contributor Author

Absolucy commented Sep 5, 2023

@itsmeow done

code/controllers/subsystem/ping.dm Outdated Show resolved Hide resolved
@itsmeow
Copy link
Member

itsmeow commented Sep 6, 2023

You have to actually call medhud update status when the afk value changes or it won't actually show.

@Absolucy
Copy link
Contributor Author

@itsmeow What'd be the best way to handle that? Since is_afk is polled, rather than being constantly checked...

Copy link
Member

@itsmeow itsmeow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@ike709 ike709 enabled auto-merge October 19, 2023 05:53
@ike709 ike709 removed the request for review from PowerfulBacon October 19, 2023 05:53
@ike709 ike709 added this pull request to the merge queue Oct 19, 2023
Merged via the queue into BeeStation:master with commit 444214b Oct 19, 2023
@Absolucy Absolucy deleted the afk-notification branch October 19, 2023 06:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Fix TGUI-Changes Contains changes to TGUI. Make sure its up to date with TGUI 4.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants