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

Fixed small UAV carrying allot of cargo. #2793

Merged
merged 1 commit into from
Oct 28, 2015

Conversation

AndreasBrostrom
Copy link
Contributor

I've fixed small UAV carrying allot of cargo.

The following parameters were active:
ace_cargo_hasCargo = 1;
ace_cargo_space = 8;

This is due to the fact that the UAV inheritance the class Helicopter

@jonpas
Copy link
Member

jonpas commented Oct 28, 2015

Can you also check UAVs which inherit from Plane, the following:

Plane
    UAV
        UAV_02_base_F
            B_UAV_02_F
            I_UAV_02_F
            O_UAV_02_F
            UAV_02_CAS_base_F
                B_UAV_02_CAS_F
                I_UAV_02_CAS_F
                O_UAV_02_CAS_F

@AndreasBrostrom
Copy link
Contributor Author

Already did that they check out and are both on
ace_cargo_hasCargo = 0;
ace_cargo_space = 0;

EDIT: Also cause Plane have 0 already.

@jonpas
Copy link
Member

jonpas commented Oct 28, 2015

Awesome 👍

Good to merge IMO.

@kymckay kymckay added the kind/bug-fix Release Notes: **FIXED:** label Oct 28, 2015
@kymckay kymckay added this to the 3.4.0 milestone Oct 28, 2015
@commy2
Copy link
Contributor

commy2 commented Oct 28, 2015

Thanks for the contribution!

commy2 added a commit that referenced this pull request Oct 28, 2015
Fixed small UAV carrying allot of cargo.
@commy2 commy2 merged commit 09ba2c8 into acemod:master Oct 28, 2015
@bux
Copy link
Member

bux commented Oct 29, 2015

Could you please add yourself to the "Authors.txt". Thanks :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug-fix Release Notes: **FIXED:**
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants