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

M249/M240 semi triggers give select fire #65883

Merged
merged 5 commits into from
Jun 7, 2023

Conversation

anoobindisguise
Copy link
Contributor

@anoobindisguise anoobindisguise commented May 27, 2023

Summary

Bugfixes "Machine gun semi auto mods don't override the default burst"

Purpose of change

The PR #53821 that introduced trigger mods for M249 and M240 references the patent https://patents.google.com/patent/US8087343B2/en which states it enables select fire, but in-game if you install the mod it actually only allows semi automatic fire.

Describe the solution

Make it select fire.
ALSO, makes the recipe need the gunsmithing prof when previously it was just welding.

Describe alternatives you've considered

Decide that it's intentional that it's semi auto only as a more improvised conversion done by the survivor rather than referencing a professionally made mechanism from a manual.

Testing

Additional context

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON <Bugfix> This is a fix for a bug (or closes open issue) astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions labels May 27, 2023
@TheShadowFerret
Copy link
Contributor

Shouldn't this be called "select fire trigger" then if it allows both 🤔

@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label May 31, 2023
@kevingranade kevingranade merged commit 6a4dd56 into CleverRaven:master Jun 7, 2023
@anoobindisguise anoobindisguise deleted the patch-20 branch June 7, 2023 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions <Bugfix> This is a fix for a bug (or closes open issue) [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants