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.

Sign up for GitHub

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

Jump to bottom

Interaction - Add action to drop distant units from group #10228

Merged
merged 5 commits into from
Aug 22, 2024
Merged

Interaction - Add action to drop distant units from group #10228

merged 5 commits into from
Aug 22, 2024

Conversation

PabstMirror
Copy link
Contributor

Adds team management action to leader to drop any unit far away
can add setting if needed (it's already under the team management settings)

@PabstMirror PabstMirror added the kind/enhancement Release Notes: **IMPROVED:** label Aug 19, 2024
@PabstMirror PabstMirror added this to the 3.18.0 milestone Aug 19, 2024
Dystopian
Dystopian reviewed Aug 19, 2024
View reviewed changes
Copy link
Contributor

@Dystopian Dystopian left a comment

Choose a reason for hiding this comment

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

The variable should be converted to number (minimal distance to drop). It can later be moved to setting as slider (-1 disable, 0..10000 distance, 100 default). Not tested.

addons/interaction/functions/fnc_canGroupDropDistantUnits.sqf Outdated Show resolved Hide resolved
addons/interaction/functions/fnc_groupDropDistantUnits.sqf Outdated Show resolved Hide resolved
@LinkIsGrim LinkIsGrim merged commit b4700ae into master Aug 22, 2024
4 checks passed
@LinkIsGrim LinkIsGrim deleted the dropUnits branch August 22, 2024 01:13
@PabstMirror PabstMirror mentioned this pull request Aug 25, 2024
Merged
@PabstMirror PabstMirror added the ignore-changelog Release Notes: Excluded label Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Dystopian Dystopian Dystopian left review comments

@LinkIsGrim LinkIsGrim LinkIsGrim approved these changes

Assignees
No one assigned
Labels
ignore-changelog Release Notes: Excluded kind/enhancement Release Notes: **IMPROVED:**
Projects
None yet
Milestone
3.18.0
Development

Successfully merging this pull request may close these issues.
3 participants
@PabstMirror @Dystopian @LinkIsGrim