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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes Shells are not appropriately handling the presence of powder bags. #2659

Merged
merged 1 commit into from
Jun 14, 2024

Conversation

IndusRobot
Copy link
Contributor

@IndusRobot IndusRobot commented May 22, 2024

About The Pull Request

Fixes a long standing issue where gunpowder is being packed directly into the shell by powder gates, allowing "pre-loading" to occur where it shouldn't via deconstruction. Feeding the shell now modifies its speed directly before it's sent to the weapon turret.

If you're loading and using the deck turret under normal conditions, nothing changes.

Why It's Good For The Game

Fixes #1385

Testing Photographs and Procedure

Screenshots&Videos
dreamseeker_KhkgwjAk7l.mp4

Since gunpowder will be held in the payload gate until used, dismantling it will return your gunpowder without modifying the shell

image

Changelog

馃啈
fix: Fixed shells are not appropriately handling the presence of powder bags
/:cl:

@Bokkiewokkie Bokkiewokkie added Active Test Merge Munitions Issues and pull requests related to munitions. labels May 24, 2024
@Bokkiewokkie Bokkiewokkie merged commit a6a4a7c into BeeStation:master Jun 14, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Active Test Merge Fix Munitions Issues and pull requests related to munitions.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Shells are not appropriately handling the presence of powder bags.
2 participants