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] Paxilmaniac makes the best skyrat pr ever | Deletes germany (every single weapon from the gunsgalore module) #1149

Merged
merged 2 commits into from
Dec 17, 2023

Conversation

Steals-The-PRs
Copy link
Collaborator

Original PR: Skyrat-SS13/Skyrat-tg#25388

About The Pull Request

As the title says simply, I am deleting every single one of the guns from the gunsgalore module.

Players will see literally zero effects of this however, because as it currently stands, these guns show up literally nowhere except the black mesa gateway we don't run.

How This Contributes To The Skyrat Roleplay Experience

Every single one of these are unused anywhere that matters, and anywhere they do "matter", they can be replaced with the now existing weapons I've made that aren't just guns from world war two. Every single one of these is an insane weapon in their own ways, too, and unknowning admins keep spawning them for events or whatever and watching as someone wipes the entire sec team with the poopenfarten 42 or whatever it is.

Proof of Testing

Er I mean look its just deleting a bunch of stuff, I could show it compiling or the guns not being in the menu or something if you REALLY wanted.

Changelog

🆑 Paxilmaniac
del: Every gun from the gunsgalore module has been deleted, long live the king
/:cl:

…ngle weapon from the gunsgalore module) (#25388)

* cant wait for all the compile this will make

* compile errors pt 1

* solfed outfit gun fixing

* fixes black mesa, again

* i missed two m16s

---------

Co-authored-by: Bloop <[email protected]>
@Steals-The-PRs Steals-The-PRs added Mirroring conflict git cherry-pick во что-то уткнулся. Не ставить вручную, только для бота Skyrat Mirror Removal labels Dec 16, 2023
@Iajret Iajret merged commit 73b3f40 into master Dec 17, 2023
22 checks passed
@Iajret Iajret deleted the upstream-mirror-25388 branch December 17, 2023 22:27
AnywayFarus added a commit that referenced this pull request Dec 17, 2023
Iajret pushed a commit that referenced this pull request Feb 26, 2024
* Minor newscaster UI fix (#81638)

## About The Pull Request

Currently newscasters note "No wanted issue posted. Have a secure day."
regardless of whether there's an active wanted issue posted or not.
Looking into the code, `Newscaster.jsx` does actually detail an
alternate message for when there _is_ an active wanted issue. It's just
only active when `wanted.active` holds true, while I believe the
`active` it's trying to access cannot be accessed without using
`wanted.map(...)`.
Making it use `wanted.map(...)` solves this.

<details>
  <summary>Image</summary>
  

![image](https://github.com/tgstation/tgstation/assets/42909981/434d1449-7663-4416-b88a-c1d89e836cd5)
  
</details>

## Why It's Good For The Game

Fixes #81600.
## Changelog
:cl:
fix: Newscasters no longer say "No wanted issue posted. Have a secure
day." when there is, in fact, an active wanted issue currently posted.
/:cl:

* Minor newscaster UI fix

---------

Co-authored-by: _0Steven <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Map Modification Mirroring conflict git cherry-pick во что-то уткнулся. Не ставить вручную, только для бота Removal Sprites
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants