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

Added GM Hire & Fixed Skill Level Options to Bulk Hire Dialog #4165

Merged
merged 2 commits into from
Jun 8, 2024

Conversation

IllianiCBT
Copy link
Collaborator

This PR adds a 'GM Hire' button to our bulk hire dialog that waives the cost of personnel hired through that dialog.

I also went ahead and added an option to generate personnel with specific skill levels. This way it's possible to hire all personnel at Elite, rather than random, skill rating.

Both of these options are only available if GM Mode is enabled.

I had wanted to extend the same functionality to 'hire minimum complement', but due to the way that menu tree is written I couldn't see an elegant way of achieving this.

- Enabled skill level selection for new personnel in 'Hire Multiple Personnel' dialog
- Added a checkbox to enable or disable skill level selection
- Updated properties file to include label for 'Fixed Skill Level' option
@IllianiCBT IllianiCBT self-assigned this Jun 7, 2024
@IllianiCBT IllianiCBT added the Personnel Personnel-related Issues label Jun 7, 2024
@IllianiCBT IllianiCBT changed the title Added GM Hire Button to Bulk Hire Dialog, Added Fixed Skill Level Option to Bulk Hire Dialog Added GM Hire Button & Fixed Skill Level Option to Bulk Hire Dialog Jun 7, 2024
@IllianiCBT IllianiCBT changed the title Added GM Hire Button & Fixed Skill Level Option to Bulk Hire Dialog Added GM Hire B& Fixed Skill Level Options to Bulk Hire Dialog Jun 7, 2024
@IllianiCBT IllianiCBT changed the title Added GM Hire B& Fixed Skill Level Options to Bulk Hire Dialog Added GM Hire & Fixed Skill Level Options to Bulk Hire Dialog Jun 7, 2024
@SJuliez SJuliez merged commit 0bef82e into MegaMek:master Jun 8, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Personnel Personnel-related Issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants