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

[TG Mirror] Heretic summon name fixes #140

Merged
merged 1 commit into from
Oct 19, 2023
Merged

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Skyrat: Skyrat-SS13/Skyrat-tg#24432
Original PR: tgstation/tgstation#79055

About The Pull Request

Fixes #79049.

The ghost poll for heretic summons used the real_name field to display the mob's name. However, for some reason, a couple of the heretic summons have jokey nicknames as their real names ("Ashy" for ash walkers, "Rusty" for rust walkers). I've opted to take the simple option of making the ghost text just use the name field instead - it's used for the admin logs around summoning, so it should be fine for the poll too.

Also, I've capitalized "Fire Shark" and set its real name to also be "Fire Shark". For consistency.

Finally, I've made heretic summon names not proper nouns, so examining them will display "That's a Rust Walker" rather than "That's Rust Walker". The Lord of the Night did not receive this treatment due to being a unique thing.

Why It's Good For The Game

It's good when ghosts know what they're actually signing up to be.

Most heretic summons aren't unique entities, so it makes sense for them to not be proper nouns.

Changelog

🆑 lizardqueenlexi
fix: Heretic summons should now display the correct name when polling ghosts to play as them.
/:cl:

* Heretic summon name fixes (#79055)

## About The Pull Request

Fixes #79049.

The ghost poll for heretic summons used the `real_name` field to display
the mob's name. However, for some reason, a couple of the heretic
summons have jokey nicknames as their real names ("Ashy" for ash
walkers, "Rusty" for rust walkers). I've opted to take the simple option
of making the ghost text just use the `name` field instead - it's used
for the admin logs around summoning, so it should be fine for the poll
too.

Also, I've capitalized "Fire Shark" and set its real name to also be
"Fire Shark". For consistency.

Finally, I've made heretic summon names not proper nouns, so examining
them will display "That's a Rust Walker" rather than "That's Rust
Walker". The Lord of the Night did not receive this treatment due to
being a unique thing.
## Why It's Good For The Game

It's good when ghosts know what they're actually signing up to be.

Most heretic summons aren't unique entities, so it makes sense for them
to not be proper nouns.
## Changelog
:cl:
fix: Heretic summons should now display the correct name when polling
ghosts to play as them.
/:cl:

* Heretic summon name fixes

---------

Co-authored-by: lizardqueenlexi <[email protected]>
@Iajret Iajret merged commit a2ac7a2 into master Oct 19, 2023
23 checks passed
@Iajret Iajret deleted the upstream-mirror-24432 branch October 19, 2023 08:31
Steals-The-PRs added a commit that referenced this pull request Oct 19, 2023
Iajret pushed a commit that referenced this pull request Dec 28, 2023
…otta other missing images (#140)

* Cache sprite data alongside spritesheets, fixing emotes and a whole lotta other missing images (#80601)

* Cache sprite data alongside spritesheets, fixing emotes and a whole lotta other missing images

* Update cleanbot_ai.dm

---------

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

Successfully merging this pull request may close these issues.

3 participants