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] Apply multiplier correctly when transferring reagents. #202

Merged
merged 1 commit into from
Oct 21, 2023

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Skyrat: Skyrat-SS13/Skyrat-tg#24489
Original PR: tgstation/tgstation#79084

About The Pull Request

  • Fixes #79083

The multiplier should be applied per reagent volume & not on the requested amount as a whole.

Changelog

🆑 SyncIt21
fix: cryo and stuff that transfers reagents with a multiplier should transfer correct volumes as expected.
/:cl:

…ts. [MDB IGNORE] (#24489)

* [NO GBP] Apply multiplier correctly when transferring reagents. (#79084)

## About The Pull Request
- Fixes #79083

The multiplier should be applied per reagent volume & not on the
requested amount as a whole.

## Changelog
:cl:
fix: cryo and stuff that transfers reagents with a multiplier should
transfer correct volumes as expected.
/:cl:

* [NO GBP] Apply multiplier correctly when transferring reagents.

---------

Co-authored-by: SyncIt21 <[email protected]>
@Iajret Iajret merged commit d8c1110 into master Oct 21, 2023
23 checks passed
@Iajret Iajret deleted the upstream-mirror-24489 branch October 21, 2023 23:36
AnywayFarus added a commit that referenced this pull request Oct 21, 2023
Iajret pushed a commit that referenced this pull request Jan 14, 2024
* Adjust Oversized quirk combat benefits

* Use signals on limb gain for Oversized quirk

* Apply suggestions from code review

Co-authored-by: Bloop <[email protected]>

* Apply suggestions from code review

Co-authored-by: Bloop <[email protected]>

* Update modular_nova/modules/oversized/code/oversized_quirk.dm

---------

Co-authored-by: Bloop <[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