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

Remove Survivor Revolvers #72638

Merged
merged 5 commits into from Mar 29, 2024

Conversation

Holli-Git
Copy link
Contributor

Summary

Bugfixes "Remove Survivor Revolvers"

Purpose of change

Survivor revolvers didn't make sense, as the cartridges lack a rim or moonclip, and a survivor wouldn't really be able to machine the specific dimensions of the cartridge into a cylinder to have it hold.

Describe the solution

Removes it.

Describe alternatives you've considered

Adding a 45 colt revolver, but then it becomes a question of would a survivor even be able to make a revolver, and so I settled on making a single shot 45 pistol in a future PR.

Testing

Tested locally, works fine.

Additional context

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Spawn Creatures, items, vehicles, locations appearing on map Items: Ammo / Guns Ammunition for all kinds of weapons and these weapons themselves <Bugfix> This is a fix for a bug (or closes open issue) astyled astyled PR, label is assigned by github actions labels Mar 26, 2024
@github-actions github-actions bot added json-styled JSON lint passed, label assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Mar 26, 2024
@TheShadowFerret
Copy link
Contributor

Adding a 45 colt revolver, but then it becomes a question of would a survivor even be able to make a revolver, and so I settled on making a single shot 45 pistol in a future PR.

Nobody has worked on the necessary tools to do so(see #66060), so the answer is "eventually, but not now so no revolver."

@Holli-Git
Copy link
Contributor Author

Adding a 45 colt revolver, but then it becomes a question of would a survivor even be able to make a revolver, and so I settled on making a single shot 45 pistol in a future PR.

Nobody has worked on the necessary tools to do so(see #66060), so the answer is "eventually, but not now so no revolver."

Yeah that's why I was going with a single shot break action pistol thing instead

@anoobindisguise
Copy link
Contributor

I'm a big fan of the idea of the survivor-made revolvers. If it's too much of a stretch to use regular ammo (and regular revolvers are common enough that you wouldn't bother anyways) why not make em like, paper cartridge guns? Those are scarcer, there aren't many M1860s lying around but they serve a valuable niche with producible ammo. The six shooter and hand cannon could pretty easily be converted to .36 and .44 paper cartridge respectively. Granted we're missing some tools we need to make them properly but I'd personally be sad to see these go with no replacement.

@Holli-Git
Copy link
Contributor Author

I'm a big fan of the idea of the survivor-made revolvers. If it's too much of a stretch to use regular ammo (and regular revolvers are common enough that you wouldn't bother anyways) why not make em like, paper cartridge guns? Those are scarcer, there aren't many M1860s lying around but they serve a valuable niche with producible ammo. The six shooter and hand cannon could pretty easily be converted to .36 and .44 paper cartridge respectively. Granted we're missing some tools we need to make them properly but I'd personally be sad to see these go with no replacement.

I dislike the notion that you need to replace something that's obsoleted, it puts extra pressure on people obsoleting things. If something doesn't make sense, you shouldn't go "But you should add a replacement!", adding a replacement can be saved for another PR. I don't want to go through the effort of researching how revolvers are made in order to make the survivor revolvers make sense, and then still not used at all because they're just not useful.

Also, do people actually use these? At all? I'm a pretty big user of niche guns and I have never gone out of my way to craft or find one of these to use them, they're just not worth it.

@Holli-Git Holli-Git marked this pull request as draft March 27, 2024 18:18
Replacing the gun with another gun causes unwanted errors, changed it to be magazines so no errors with gun being migrated to a different caliber.
@Holli-Git Holli-Git marked this pull request as ready for review March 27, 2024 18:33
@Maleclypse
Copy link
Member

I'm a big fan of the idea of the survivor-made revolvers. If it's too much of a stretch to use regular ammo (and regular revolvers are common enough that you wouldn't bother anyways) why not make em like, paper cartridge guns? Those are scarcer, there aren't many M1860s lying around but they serve a valuable niche with producible ammo. The six shooter and hand cannon could pretty easily be converted to .36 and .44 paper cartridge respectively. Granted we're missing some tools we need to make them properly but I'd personally be sad to see these go with no replacement.

I dislike the notion that you need to replace something that's obsoleted, it puts extra pressure on people obsoleting things. If something doesn't make sense, you shouldn't go "But you should add a replacement!", adding a replacement can be saved for another PR. I don't want to go through the effort of researching how revolvers are made in order to make the survivor revolvers make sense, and then still not used at all because they're just not useful.

I do have to agree that it is unfortunately on the people who want a realistic way to make a thing to do the work of getting it back in game. On occasion for something requiring massive c++ work the dev team goes the opposite direction, see UPS and repair kits. But as this and everything required to make survivor revolvers possible is doable in json it wouldn’t be waived here.

@Maleclypse Maleclypse merged commit 64e1db1 into CleverRaven:master Mar 29, 2024
21 of 24 checks passed
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) Items: Ammo / Guns Ammunition for all kinds of weapons and these weapons themselves [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions Spawn Creatures, items, vehicles, locations appearing on map
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants