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 Ability to Tell autoAwards to Ignore Individual Custom Award Sets #4493

Merged
merged 2 commits into from
Aug 2, 2024

Conversation

IllianiCBT
Copy link
Collaborator

This PR introduces another long requested feature: the ability to filter award sets so that autoAwards won't use specific custom award sets. This is as simple as entering the names of the award sets you don't want autoAwards to use into a free text box in campaign options. As it runs autoAwards will check whether an award set appears in that list, and if it does it will ignore all awards from that set.

Introduced a new field `awardSetFilterList` in `CampaignOptions` for filtering award sets in autoAwards. Updated related GUI components and serialization methods to manage this new option. Enhanced `AutoAwardsController` to respect the filter list during award set processing.
@IllianiCBT IllianiCBT added Awards UX User experience labels Jul 27, 2024
@IllianiCBT IllianiCBT self-assigned this Jul 27, 2024
@codecov-commenter
Copy link

codecov-commenter commented Jul 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 10.31%. Comparing base (e4213fe) to head (c9d05e5).
Report is 1 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #4493      +/-   ##
============================================
- Coverage     10.31%   10.31%   -0.01%     
- Complexity     5813     5814       +1     
============================================
  Files           934      934              
  Lines        128441   128474      +33     
  Branches      18910    18912       +2     
============================================
+ Hits          13253    13256       +3     
- Misses       113912   113942      +30     
  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 1df4954 into MegaMek:master Aug 2, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awards UX User experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants