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

Bullet Stack Fixes + Other #3568

Merged
merged 12 commits into from
Oct 26, 2024

Conversation

generalthrax
Copy link
Contributor

@generalthrax generalthrax commented Oct 15, 2024

About The Pull Request

  • Fixes .299 Eoehoma being 5 per stack instead of 15
  • Fixes 5.56x42 CLIP being 5 per stack instead of 15
  • Rebalanced the tinyass .22lr bullet stacks to be 25 instead of 15, for 100 rounds per box
  • Changed the ammo can to be bulky and lets it hold 4 ammo boxes, fixes pre-spawned ammo cans to actually fit ammo boxes
  • Ammo cans can only hold ammo and explosives
  • Fixed a few icons and descriptions for ammo boxes
  • Fixed 12g beanbag to actually have beanbag inside / along with 12g rubbershot
  • Increased the speed of reloading until we get loading benches
  • .22 is out of the black market since its in cargo

Why It's Good For The Game

Fixes a bunch of stuff leftover from Bulletstacks. Rebalanced a few ammostacks to actually be usable with their guns. You will no longer have to stand in place for 3 minutes to reload a single Pounder panmag. Bug fixes good.

Changelog

🆑
balance: .22lr is now 25 per handful instead of 12
balance: Reloading speed from stacks doubled
balance: Increase ammo can to bulky and lets it hold 4 normal-sized ammoboxes, can only hold ammo and explosives.
balance: Adjusted price of .299 Eoehoma slightly, adds a second box to the cargo order since its 60 instead of 120
fix: Fixes a bunch of bullet stacks to have correct numbers
fix: Larger handfuls (.22lr)
fix: Mapped in ammo cans now hold their boxes
fix: 12g beanbag now has beanbag instead of rubber + beanbag
fix: Ammo box icons and descriptions
del: Removed .22lr from the blackmarket. Its in cargo already.
/:cl:

@github-actions github-actions bot added the Code change Watch something violently break. label Oct 15, 2024
@FalloutFalcon FalloutFalcon added the Test Merge Candidate You do it for free. label Oct 16, 2024
Comment on lines +19 to +28
STR.set_holdable(list(
/obj/item/storage/box/ammo,
/obj/item/mine,
/obj/item/grenade,
/obj/item/ammo_casing/caseless/rocket,
/obj/item/ammo_box/magazine/ammo_stack,
/obj/item/ammo_casing,
/obj/item/mine,
/obj/item/grenade
))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would go ahead and make this a var because sub types always end up needed edge cases for this.

@FalloutFalcon FalloutFalcon removed the Test Merge Candidate You do it for free. label Oct 16, 2024
@rye-rice
Copy link
Member

needs another TM?

@generalthrax
Copy link
Contributor Author

needs another TM?

Should be ready? Question mark?

Copy link
Member

@thgvr thgvr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, just a few worries

code/game/objects/items/storage/ammo_can.dm Show resolved Hide resolved
@@ -490,7 +492,7 @@
/obj/item/storage/box/ammo/a44roum_hp
name = "box of HP .44 roumain ammo"
desc = "A box of .44 roumain rubbershot ammo, designed to disable targets without causing serious damage."
icon_state = "a44roum-rubber"
icon_state = "a44roum-hp"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that doesn't seem right. is there no sprite for -rubber here?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't rubber. This is hollow point.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ooprs

@thgvr thgvr added this pull request to the merge queue Oct 26, 2024
@FalloutFalcon FalloutFalcon removed this pull request from the merge queue due to a manual request Oct 26, 2024
@SomeguyManperson SomeguyManperson added this pull request to the merge queue Oct 26, 2024
Merged via the queue into shiptest-ss13:master with commit 06092d4 Oct 26, 2024
16 checks passed
zimon9 pushed a commit to zimon9/Shiptest-PR-testing that referenced this pull request Oct 29, 2024
## About The Pull Request

- Fixes .299 Eoehoma being 5 per stack instead of 15
- Fixes 5.56x42 CLIP being 5 per stack instead of 15
- Rebalanced the tinyass .22lr bullet stacks to be 25 instead of 15, for
100 rounds per box
- Changed the ammo can to be bulky and lets it hold 4 ammo boxes, fixes
pre-spawned ammo cans to actually fit ammo boxes
- Ammo cans can only hold ammo and explosives
- Fixed a few icons and descriptions for ammo boxes
- Fixed 12g beanbag to actually have beanbag inside / along with 12g
rubbershot
- Increased the speed of reloading until we get loading benches
- .22 is out of the black market since its in cargo

## Why It's Good For The Game

Fixes a bunch of stuff leftover from Bulletstacks. Rebalanced a few
ammostacks to actually be usable with their guns. You will no longer
have to stand in place for 3 minutes to reload a single Pounder panmag.
Bug fixes good.

## Changelog

:cl:
balance: .22lr is now 25 per handful instead of 12
balance: Reloading speed from stacks doubled
balance: Increase ammo can to bulky and lets it hold 4 normal-sized
ammoboxes, can only hold ammo and explosives.
balance: Adjusted price of .299 Eoehoma slightly, adds a second box to
the cargo order since its 60 instead of 120
fix: Fixes a bunch of bullet stacks to have correct numbers
fix: Larger handfuls (.22lr)
fix: Mapped in ammo cans now hold their boxes
fix: 12g beanbag now has beanbag instead of rubber + beanbag
fix: Ammo box icons and descriptions
del: Removed .22lr from the blackmarket. Its in cargo already.
/:cl:
zimon9 pushed a commit to zimon9/Shiptest-PR-testing that referenced this pull request Oct 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code change Watch something violently break.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants