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] Spawners menu now displays the amount of uses left appropriately #2022

Merged
merged 1 commit into from
Feb 17, 2024

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#1006
Original PR: tgstation/tgstation#81496

About The Pull Request

Prior to this PR, if an individual spawner allowed more than one uses, it would still display as one use in the menu. Not only that, but infinite spawners would also show as having just one use. Both of these behaviors have been fixed, resulting in a much more accurate depiction of how many uses are left on each spawner.

image
(The first spawner isn't normally infinite, I just used VV to edit its infinite_uses to 1 for the sake of proving that the display worked).

Why It's Good For The Game

Being able to know that a spawner is infinite is a good thing for players, just like being able to know how many uses are left if an individual spawner can be used by more than one player.

Changelog

🆑 GoldenAlpharex
fix: The Spawners menu now accurately displays the amount of uses left in each spawner option, taking into account individual spawners that either allow more than one use, or an infinite amount of uses.
/:cl:

…tely (#1006)

* Spawners menu now displays the amount of uses left appropriately (#81496)

## About The Pull Request
Prior to this PR, if an individual spawner allowed more than one uses,
it would still display as one use in the menu. Not only that, but
infinite spawners would also show as having just one use. Both of these
behaviors have been fixed, resulting in a much more accurate depiction
of how many uses are left on each spawner.


![image](https://github.com/tgstation/tgstation/assets/58045821/77508cdd-66bf-4d85-9dd7-c25f8ccd41b6)
(The first spawner isn't normally infinite, I just used VV to edit its
`infinite_uses` to `1` for the sake of proving that the display worked).


## Why It's Good For The Game
Being able to know that a spawner is infinite is a good thing for
players, just like being able to know how many uses are left if an
individual spawner can be used by more than one player.

## Changelog

:cl: GoldenAlpharex
fix: The Spawners menu now accurately displays the amount of uses left
in each spawner option, taking into account individual spawners that
either allow more than one use, or an infinite amount of uses.
/:cl:

* Spawners menu now displays the amount of uses left appropriately

---------

Co-authored-by: GoldenAlpharex <58045821+GoldenAlpharex@users.noreply.github.com>
@Iajret Iajret merged commit 2843895 into master Feb 17, 2024
24 checks passed
@Iajret Iajret deleted the upstream-mirror-1006 branch February 17, 2024 16:21
AnywayFarus added a commit that referenced this pull request Feb 17, 2024
Iajret pushed a commit that referenced this pull request Apr 16, 2024
* Birdshot: engi wardrope. (#82639)

## About The Pull Request

Add engi wardrope on Birdshot.

## Why It's Good For The Game

Birdshot doesn't have engi wardrope.

:cl:
fix: Birdshot now have engi wardrope
/:cl:

* Birdshot: engi wardrope.

---------

Co-authored-by: Xackii <120736708+Xackii@users.noreply.github.com>
ReezeBL pushed a commit that referenced this pull request Apr 16, 2024
* Birdshot: engi wardrope. (#82639)

## About The Pull Request

Add engi wardrope on Birdshot.

## Why It's Good For The Game

Birdshot doesn't have engi wardrope.

:cl:
fix: Birdshot now have engi wardrope
/:cl:

* Birdshot: engi wardrope.

---------

Co-authored-by: NovaBot <154629622+NovaBot13@users.noreply.github.com>
Co-authored-by: Xackii <120736708+Xackii@users.noreply.github.com>
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