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

Update setUnitLoadout to CBA_fnc_setLoadout #920

Merged
merged 5 commits into from
Nov 19, 2023

Conversation

Thatguy553
Copy link

@Thatguy553 Thatguy553 commented Oct 30, 2022

  • Updated open_arsenal.sqf and redeploy_manager.sqf to use CBA_fnc_setLoadout.

  • Created config file bool variable to allow easy change of CBA_fnc_setLoadout's third param to refill mags.

Q A
Bug fix? yes
New feature? no
Needs wipe? no

Description:

Basically updated the vanilla setUnitLoadout used for ace loadouts to instead use CBA_fnc_setLoadout.

Content:

  • Fix error causing new ACE loadouts to no longer be compatible with liberation.

Successfully tested on:

  • Local MP
  • Dedicated MP

- Updated open_arsenal.sqf and redeploy_manager.sqf to use CBA_fnc_setLoadout.

- Created config file bool variable to allow easy change of CBA_fnc_setLoadout's third param to refill mags.
@TACHarsis
Copy link

Please merge this!

Apricot-ale added a commit to Apricot-ale/KP-Liberation-APR that referenced this pull request Feb 28, 2023
Apricot-ale added a commit to Apricot-ale/KP-Liberation-APR that referenced this pull request Mar 2, 2023
Copy link
Collaborator

@some-evil-kitty some-evil-kitty left a comment

Choose a reason for hiding this comment

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

Tested, looks good

@some-evil-kitty some-evil-kitty changed the base branch from master to v0.96.8 November 19, 2023 19:26
some-evil-kitty

This comment was marked as duplicate.

Copy link
Member

@veteran29 veteran29 left a comment

Choose a reason for hiding this comment

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

Code LGTM

@TACHarsis
Copy link

Please merge this!

Just took a year 😌

@some-evil-kitty some-evil-kitty merged commit 912a035 into KillahPotatoes:v0.96.8 Nov 19, 2023
1 check passed
Crowdedlight added a commit to Crowdedlight/KP-Liberation-war-room that referenced this pull request Mar 4, 2024
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.

4 participants