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] Fixing Cargo (Take 2) #2468

Merged
merged 1 commit into from
Mar 19, 2024
Merged

[MIRROR] Fixing Cargo (Take 2) #2468

merged 1 commit into from
Mar 19, 2024

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#1512
Original PR: tgstation/tgstation#82079

About The Pull Request

  • Fixes #82070

All exports are broken because we created a new report for every time sold & didn't reuse the one given to us.

It works now

Screenshot (405)

Changelog

🆑 SyncIt21
fix: Selling stuff in cargo gives us money again.
/:cl:

* Fixing Cargo (Take 2) (#82079)

## About The Pull Request
- Fixes #82070

All exports are broken because we created a new report for every time
sold & didn't reuse the one given to us.

It works now

![Screenshot
(405)](https://github.com/tgstation/tgstation/assets/110812394/26aa9276-c5d6-40f7-ade3-d62d07486cdd)

## Changelog
:cl:
fix: Selling stuff in cargo gives us money again.
/:cl:

* Fixing Cargo (Take 2)

---------

Co-authored-by: SyncIt21 <[email protected]>
@Iajret Iajret merged commit 5e61eb1 into master Mar 19, 2024
24 checks passed
@Iajret Iajret deleted the upstream-mirror-1512 branch March 19, 2024 08:56
AnywayFarus added a commit that referenced this pull request Mar 19, 2024
Iajret pushed a commit that referenced this pull request May 15, 2024
* [NO GBP] Fixes the knife offsets of pipeguns (#83155)

<!-- Write **BELOW** The Headers and **ABOVE** The comments else it may
not be viewable. -->
<!-- You can view Contributing.MD for a detailed description of the pull
request process. -->

## About The Pull Request

Fixes the knife offsets of pipeguns.

## Why It's Good For The Game

I completely forgot to fix this when I updated the pipegun sprites.
Oops.

## Changelog

<!-- If your PR modifies aspects of the game that can be concretely
observed by players or admins you should add a changelog. If your change
does NOT meet this description, remove this section. Be sure to properly
mark your PRs to prevent unnecessary GBP loss. You can read up on GBP
and it's effects on PRs in the tgstation guides for contributors. Please
note that maintainers freely reserve the right to remove and add tags
should they deem it appropriate. You can attempt to finagle the system
all you want, but it's best to shoot for clear communication right off
the bat. -->

:cl:
fix: Fixes the knife offset for pipeguns
/:cl:

<!-- Both :cl:'s are required for the changelog to work! You can put
your name to the right of the first :cl: if you want to overwrite your
GitHub username as author ingame. -->
<!-- You can use multiple of the same prefix (they're only used for the
icon ingame) and delete the unneeded ones. Despite some of the tags,
changelogs should generally represent how a player might be affected by
the changes rather than a summary of the PR's contents. -->

* [NO GBP] Fixes the knife offsets of pipeguns

---------

Co-authored-by: necromanceranne <[email protected]>
Co-authored-by: NovaBot13 <[email protected]>
ReezeBL pushed a commit that referenced this pull request May 15, 2024
* [NO GBP] Fixes the knife offsets of pipeguns (#83155)

<!-- Write **BELOW** The Headers and **ABOVE** The comments else it may
not be viewable. -->
<!-- You can view Contributing.MD for a detailed description of the pull
request process. -->

## About The Pull Request

Fixes the knife offsets of pipeguns.

## Why It's Good For The Game

I completely forgot to fix this when I updated the pipegun sprites.
Oops.

## Changelog

<!-- If your PR modifies aspects of the game that can be concretely
observed by players or admins you should add a changelog. If your change
does NOT meet this description, remove this section. Be sure to properly
mark your PRs to prevent unnecessary GBP loss. You can read up on GBP
and it's effects on PRs in the tgstation guides for contributors. Please
note that maintainers freely reserve the right to remove and add tags
should they deem it appropriate. You can attempt to finagle the system
all you want, but it's best to shoot for clear communication right off
the bat. -->

:cl:
fix: Fixes the knife offset for pipeguns
/:cl:

<!-- Both :cl:'s are required for the changelog to work! You can put
your name to the right of the first :cl: if you want to overwrite your
GitHub username as author ingame. -->
<!-- You can use multiple of the same prefix (they're only used for the
icon ingame) and delete the unneeded ones. Despite some of the tags,
changelogs should generally represent how a player might be affected by
the changes rather than a summary of the PR's contents. -->

* [NO GBP] Fixes the knife offsets of pipeguns

---------

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