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] Hallucinations don't print to newscasters #639

Merged
merged 1 commit into from
Nov 15, 2023

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Skyrat: Skyrat-SS13/Skyrat-tg#25011
Original PR: tgstation/tgstation#79711

About The Pull Request

Fixes #79709

I thought it was clever to use priority_announce for the hallucinations, as means for maintaining parity, unfortunately the proc is not pure and prints reports to the newscasters. Oopsie.

This makes it so only reports sent to all players are printed to newscasters.

Changelog

🆑 Melbert
fix: Fixes hallucination and encrypted announcements printing to the Newscaster.
/:cl:

* Hallucinations don't print to newscasters

* Update priority_announce.dm

---------

Co-authored-by: MrMelbert <[email protected]>
Co-authored-by: Giz <[email protected]>
@Iajret Iajret merged commit 7319d48 into master Nov 15, 2023
24 checks passed
@Iajret Iajret deleted the upstream-mirror-25011 branch November 15, 2023 09:41
AnywayFarus added a commit that referenced this pull request Nov 15, 2023
Iajret pushed a commit that referenced this pull request Jan 27, 2024
* New Telecommunications Monitoring Console UI (#81111)

I've made a new UI for Telecommunications Monitoring Console. New
features include: more useful information, search bar, icons and smaller
window. This PR also includes a new sprite sheet, be warned!

Additionally, I've removed `updateUsrDialog` because it makes no sense
here and UI updates are handled by TGUI.

* New Telecommunications Monitoring Console UI

---------

Co-authored-by: Interception&? <[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