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] Buying the advanced plastic surgery disk now gives you the right disk #2214

Merged
merged 1 commit into from
Mar 2, 2024

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#1226
Original PR: tgstation/tgstation#81746

About The Pull Request

Buying the advanced plastic surgery disk currently gives you a brainwashing disk instead, this fixes that.

Why It's Good For The Game

big oversight, you now get the right program you bought.

Changelog

🆑 JohnFulpWillard
fix: Buying the advanced plastic surgery disk from the uplink now gives you advanced plastic surgery instead of brainwashing.
/:cl:

…ight disk (#1226)

* Buying the advanced plastic surgery disk now gives you the right disk (#81746)

## About The Pull Request

Buying the advanced plastic surgery disk currently gives you a
brainwashing disk instead, this fixes that.

## Why It's Good For The Game

big oversight, you now get the right program you bought.

## Changelog

:cl:
fix: Buying the advanced plastic surgery disk from the uplink now gives
you advanced plastic surgery instead of brainwashing.
/:cl:

* Buying the advanced plastic surgery disk now gives you the right disk

---------

Co-authored-by: John Willard <[email protected]>
@Iajret Iajret merged commit 69da62c into master Mar 2, 2024
24 checks passed
@Iajret Iajret deleted the upstream-mirror-1226 branch March 2, 2024 19:16
AnywayFarus added a commit that referenced this pull request Mar 2, 2024
Iajret pushed a commit that referenced this pull request Apr 28, 2024
)

* Removes examine text with Med/Sec HUDs being under Radio (#82899)

## About The Pull Request

As title says
One of these spans didn't have a closing span so this fixes that too i
guess.

These are all spans that would include your message under radio
``'.alert, .minorannounce, .syndradio, .centcomradio, .aiprivradio,
.comradio, .secradio, .gangradio, .engradio, .medradio, .sciradio,
.suppradio, .servradio, .radio, .deptradio, .binarysay, .newscaster,
.resonate, .abductor, .alien, .changeling'``

To fix it I just removed the span since it did nothing anyway.
I didn't remove it in case it's ever brought back in the future and for
any downstream that might use it, it feels off to remove chat constants.

It also looks no different either way

![image](https://github.com/tgstation/tgstation/assets/53777086/46673645-4ab4-4bd2-9a9f-a26b19aa0e87)

tried with multiple fonts, this really has no effect it's purely for
categorization.

## Why It's Good For The Game

Closes tgstation/tgstation#72487

## Changelog

:cl:
fix: Examining someone with Med/Sec HUDs will no longer filter the
message under Radio.
/:cl:

* Removes examine text with Med/Sec HUDs being under Radio

---------

Co-authored-by: John Willard <[email protected]>
ReezeBL pushed a commit that referenced this pull request Apr 29, 2024
) (#3092)

* Removes examine text with Med/Sec HUDs being under Radio (#82899)

## About The Pull Request

As title says
One of these spans didn't have a closing span so this fixes that too i
guess.

These are all spans that would include your message under radio
``'.alert, .minorannounce, .syndradio, .centcomradio, .aiprivradio,
.comradio, .secradio, .gangradio, .engradio, .medradio, .sciradio,
.suppradio, .servradio, .radio, .deptradio, .binarysay, .newscaster,
.resonate, .abductor, .alien, .changeling'``

To fix it I just removed the span since it did nothing anyway.
I didn't remove it in case it's ever brought back in the future and for
any downstream that might use it, it feels off to remove chat constants.

It also looks no different either way

![image](https://github.com/tgstation/tgstation/assets/53777086/46673645-4ab4-4bd2-9a9f-a26b19aa0e87)

tried with multiple fonts, this really has no effect it's purely for
categorization.

## Why It's Good For The Game

Closes tgstation/tgstation#72487

## Changelog

:cl:
fix: Examining someone with Med/Sec HUDs will no longer filter the
message under Radio.
/:cl:

* Removes examine text with Med/Sec HUDs being under Radio

---------

Co-authored-by: NovaBot <[email protected]>
Co-authored-by: John Willard <[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