Skip to content

PR: MekHQ Client Options to not fire surplus temp personnel on new day#9099

Merged
IllianiBird merged 1 commit into
MegaMek:mainfrom
psikomonkie:pr/personnel/dont-fire-surplus
May 26, 2026
Merged

PR: MekHQ Client Options to not fire surplus temp personnel on new day#9099
IllianiBird merged 1 commit into
MegaMek:mainfrom
psikomonkie:pr/personnel/dont-fire-surplus

Conversation

@psikomonkie
Copy link
Copy Markdown
Member

The MekHQ client options for hiring temp personnel on a new day always technically reset the temp personnel - it would fire surplus.

When mothballing units with temp crew, the temp crew will be fired if "fill temp crew on new day" is enabled.

This adds a setting for each option to not fire surplus personnel but still hire to get past any shortfalls and auto assign to units.

image

@psikomonkie psikomonkie requested a review from a team as a code owner May 25, 2026 21:55
@codecov
Copy link
Copy Markdown

codecov Bot commented May 25, 2026

Codecov Report

❌ Patch coverage is 0% with 171 lines in your changes missing coverage. Please review.
✅ Project coverage is 15.18%. Comparing base (a361ba0) to head (234164c).
⚠️ Report is 33 commits behind head on main.

Files with missing lines Patch % Lines
MekHQ/src/mekhq/gui/dialog/MHQOptionsDialog.java 0.00% 100 Missing ⚠️
...ekHQ/src/mekhq/campaign/CampaignNewDayManager.java 0.00% 41 Missing ⚠️
MekHQ/src/mekhq/MHQOptions.java 0.00% 30 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #9099      +/-   ##
============================================
- Coverage     15.23%   15.18%   -0.05%     
  Complexity     9305     9305              
============================================
  Files          1304     1305       +1     
  Lines        172518   173071     +553     
  Branches      26063    26146      +83     
============================================
+ Hits          26277    26278       +1     
- Misses       143198   143752     +554     
+ Partials       3043     3041       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

chkNewDayVesselCrewPoolFill.text=Fill Vessel Crew Pool
chkNewDayVesselCrewPoolFill.toolTipText=Automatically fill the Vessel Crew pool immediately upon selecting new day,\
\ and distribute to vessel units.
chkNewDayAstechPoolNoRelease.text=└ Do Not Release Surplus Astechs
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this really the cleanest way to mark dependent checkboxes?
Just when I thought my opinion of AWT/Swing couldn't get any lower -_-;

Copy link
Copy Markdown
Collaborator

@Sleet01 Sleet01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@IllianiBird IllianiBird merged commit 1e28427 into MegaMek:main May 26, 2026
8 checks passed
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.

3 participants