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.

Sign up for GitHub

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

Jump to bottom

SecHUD flags not showing on examine with sechud #18479

Closed
avanth89 opened this issue Apr 13, 2024 · 2 comments · Fixed by #18486
Closed

SecHUD flags not showing on examine with sechud #18479

avanth89 opened this issue Apr 13, 2024 · 2 comments · Fixed by #18486
Labels
C-Bug A bug that impacts usage of a feature S-Possibly-Intended A bug covered by a bug report or fix PR may be intended, and needs dev feedback

Comments

@avanth89
Copy link
Contributor

Labels

[BUG]

Description

SecHUD flags not showing on examine.

Steps to reproduce

  1. Set someones secflag in secmate.
  2. Examine someone.
  3. No secflag shown on examine.

Expected Behavior

SecHUD flags to show on examine.

Additional Information & Screenshots

Marked several people during a round. Not a single one had their SecHUD flag showing on an examine.

Screenshot 2024-04-12 182609
Screenshot 2024-04-12 182617
@keywordlabeler keywordlabeler bot added the C-Bug A bug that impacts usage of a feature label Apr 13, 2024
@pgmzeta
Copy link
Contributor

pgmzeta commented Apr 13, 2024

SecHUD flags currently only show when the person has a criminal status set, but I don't think it is communicated anywhere.

@pgmzeta pgmzeta added the S-Possibly-Intended A bug covered by a bug report or fix PR may be intended, and needs dev feedback label Apr 13, 2024
@pgmzeta pgmzeta mentioned this issue Apr 13, 2024
Merged
@avanth89
Copy link
Contributor Author

Might not be, but I'm fairly certain I had tested after flagging as well. I can run a local later to double check!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-Bug A bug that impacts usage of a feature S-Possibly-Intended A bug covered by a bug report or fix PR may be intended, and needs dev feedback
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

See SecHUD Flag regardless of arrest icon status pgmzeta/goonstation
2 participants
@avanth89 @pgmzeta