Skip to content

fixes an attachment bug #9286

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Detective-Google
Copy link
Contributor

@Detective-Google Detective-Google commented May 17, 2025

About the pull request

fixes swapping to an underbarrel attachment resetting a weapon's fire cd by effectively telling the gun it has "fired" whenever the swap happens and starting a new CD
closes #9285

Explain why it's good for the game

not a very elegant solution but I hate it less than removing items because they have a bug I haven't been able to track down

Testing Photographs and Procedure

Screenshots & Videos

pinky promise I tested it for a couple minutes in local.

Changelog

🆑
fix:attached guns no longer cause gun fire delay issues
/:cl:

@github-project-automation github-project-automation bot moved this to Awaiting Review in Review Backlog May 17, 2025
@cmss13-ci cmss13-ci bot added the Fix Fix one bug, make ten more label May 17, 2025
@cm13-github
Copy link
Contributor

cm13-github commented May 17, 2025

Test merge deployment history:

Test Merge Deployed @ 05/17/2025 02:54:02 +00:00:
Server Instance

CM13 Live

Revision

Origin: 793853c
Pull Request: a34219c
Server: 2e1151c

Test Merge Updated @ 05/17/2025 13:26:59 +00:00:
Server Instance

CM13 Live

Revision

Origin: ec1c2c0
Pull Request: a34219c
Server: 5065773a20b17d6b79de6f28cb49f65680040bbb

Test Merge Updated @ 05/17/2025 17:47:57 +00:00:
Server Instance

CM13 Live

Revision

Origin: ec1c2c0
Pull Request: a34219c
Server: 15222cb

Test Merge Updated @ 05/17/2025 19:42:35 +00:00:
Server Instance

CM13 Live

Revision

Origin: ec1c2c0
Pull Request: a34219c
Server: b2e1bd8

Test Merge Updated @ 05/18/2025 01:59:54 +00:00:
Server Instance

CM13 Live

Revision

Origin: 04612c1
Pull Request: a34219c
Server: 162704396e432639108eb724f4ca8da02f3dfc00

Test Merge Updated @ 05/18/2025 13:02:53 +00:00:
Server Instance

CM13 Live

Revision

Origin: c1de657
Pull Request: a34219c
Server: dda4f8f7d6ecdcbdd286944ffac9199fc94243b8

Test Merge Updated @ 05/18/2025 16:29:40 +00:00:
Server Instance

CM13 Live

Revision

Origin: c1de657
Pull Request: a34219c
Server: 9cf719f

Test Merge Updated @ 05/18/2025 17:36:34 +00:00:
Server Instance

CM13 Live

Revision

Origin: c1de657
Pull Request: a34219c
Server: cbbe64b

Test Merge Updated @ 05/19/2025 02:05:43 +00:00:
Server Instance

CM13 Live

Revision

Origin: fb88d2d
Pull Request: a34219c
Server: d0bf7912ba9092c27aa2f907bfbed755895d98b9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Fix one bug, make ten more
Projects
Status: Awaiting Review
Development

Successfully merging this pull request may close these issues.

2 participants