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 DropShip Bounty Settings #4558

Merged
merged 4 commits into from
Aug 8, 2024
Merged

Conversation

IllianiCBT
Copy link
Collaborator

Requires MegaMek/megamek#5862

Introduced a new campaign option to award a percentage bonus for destroyed DropShips. This bonus can be configured in the campaign settings and (if enabled) will replace the salvaging of DropShips killed on ground maps with a set percentage of the unit's sale value.

In universe, this represents the difficulty of salvaging large vessels from the field, the salvage site instead being sold to an interested third party.

This was introduced to offer an alternative to the sudden finance spike experienced when killing DropShips. Or, if the employer is allowed to salvage the unit, the ability to suddenly be able to salvage everything until the end of the contract.

Personally, I find a setting of 5% works best, providing a nice boost to finances without derailing the economic/equipment growth of the campaign.

Closes #3577

Introduced a new option to award a percentage bonus for destroyed DropShips in campaigns. This bonus can be configured in the campaign settings and will replace the salvage value for DropShips, representing the difficulty of removing large vessels from the field.
Updated the spinner's maximum value for DropShip bonus percentage from 100 to 20. This change better aligns with the intended game mechanics and balance.
@Thom293
Copy link
Contributor

Thom293 commented Aug 6, 2024

most excellent!

IllianiCBT and others added 2 commits August 7, 2024 14:01
Clarified the tooltip for the DropShip bonus percentage option. The new description specifies that setting the option above 0 removes the ability to salvage DropShips from ground maps and provides a bounty instead.
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 10.32%. Comparing base (b5ff179) to head (f24d895).
Report is 6 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #4558      +/-   ##
============================================
- Coverage     10.32%   10.32%   -0.01%     
- Complexity     5816     5817       +1     
============================================
  Files           935      935              
  Lines        128792   128835      +43     
  Branches      18951    18955       +4     
============================================
+ Hits          13296    13299       +3     
- Misses       114220   114260      +40     
  Partials       1276     1276              

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

Copy link
Member

@HammerGS HammerGS left a comment

Choose a reason for hiding this comment

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

Tested and seems to work

@IllianiCBT IllianiCBT merged commit 51b8d11 into MegaMek:master Aug 8, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RFE: Campaign Option to convert salvaged Dropships to Bounty
4 participants