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

Coolant Pods equipment category fix #1354

Merged
merged 2 commits into from
Dec 29, 2023

Conversation

SJuliez
Copy link
Member

@SJuliez SJuliez commented Dec 29, 2023

Fixes #1353

Coolant Pods now show up in "Other", not "Ammo" and are no longer influenced by the "unusable ammo" filter

@redbatz
Copy link
Contributor

redbatz commented Dec 29, 2023

The check at line 94 can probably be removed with this change, it was added from my previous PR #1027

@SJuliez
Copy link
Member Author

SJuliez commented Dec 29, 2023

Yes indeed, thanks

@HammerGS
Copy link
Member

Works for me.

@SJuliez SJuliez merged commit 1940af6 into MegaMek:master Dec 29, 2023
3 of 4 checks passed
@SJuliez SJuliez deleted the coolant-pod-category branch March 9, 2024 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Change made it so Coolant Pods not shown in Equipment View unless hide option is disabled
3 participants