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

[MIRROR] Allows Signers to send COMSIG_MOB_SAY #612

Merged
merged 1 commit into from
Nov 14, 2023

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Skyrat: Skyrat-SS13/Skyrat-tg#24982
Original PR: tgstation/tgstation#79677

About The Pull Request

By removing a conditional statement which prevented signers from sending a COMSIG_MOB_SAY signal (for some reason), this PR fixes a few things in one go:

  • Sign language once again prints a notice that the signer raises or lowers their eyebrows for exclamations and questions respectively.
  • Tonal indicators (sign/typing icons that depict eyebrows raising or lowering) are also working again.
  • RuneChat is now punctuated the same as the chat box for Signers.

This PR is now atomic thanks to @ MrMelbert.

Why It's Good For The Game

Repairs intended functionality that broke a while ago. Adds a little flavor and spice to sign conversation again. Makes RuneChat properly register Signer punctuation.
signer

Changelog

🆑 Danny Boy
fix: Fixed Signer eyebrow raising/lowering indicators and emotes
fix: Fixed Signer RuneChat punctuation
/:cl:

* Allows Signers to send COMSIG_MOB_SAY (#79677)

## About The Pull Request
By removing a conditional statement which prevented signers from sending
a `COMSIG_MOB_SAY` signal (for some reason), this PR fixes a few things
in one go:

- Sign language once again prints a notice that the signer raises or
lowers their eyebrows for exclamations and questions respectively.
- Tonal indicators (sign/typing icons that depict eyebrows raising or
lowering) are also working again.
- RuneChat is now punctuated the same as the chat box for Signers.

This PR is now atomic thanks to @ MrMelbert.

* Allows Signers to send COMSIG_MOB_SAY

---------

Co-authored-by: DBGit42 <[email protected]>
@ReezeBL ReezeBL merged commit 28ae35f into master Nov 14, 2023
24 checks passed
@ReezeBL ReezeBL deleted the upstream-mirror-24982 branch November 14, 2023 08:01
AnywayFarus added a commit that referenced this pull request Nov 14, 2023
Iajret pushed a commit that referenced this pull request Jan 25, 2024
* [GBP NO UPDATE] Hub time timezone fix (#81053)

## About The Pull Request
While browsing the hub I noticed Psychonaut Station the Turkish TG
always had "Time:" with a value of many hours. I thought they had long
rounds until I joined the server and noticed the actual time was 3 hours
behind

Turns out time2text() has a timezone arg which defaults to server
timezone

I have set it to UTC+0 so it is always consistent since world.time is
UTC
## Changelog
:cl:
fix: The hub time should be accurate for servers with different
timezones
/:cl:

* [GBP NO UPDATE] Hub time timezone fix

---------

Co-authored-by: 13spacemen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants