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

Add ability to create larger supply boxes with smaller resupply boxes inside them #540

Merged
merged 4 commits into from
Sep 20, 2024

Conversation

lambdatiger
Copy link
Contributor

@lambdatiger lambdatiger commented Sep 7, 2024

This adds an idea kilo had/stole in the resupply boxes suggestion on BW's discord. It allows a mission maker to use the CfgLoadouts SupplyBoxes class to make larger boxes with multiple smaller resupply boxes inside them. See this example

class CfgLoadouts {
  // other code before or after
  // Do Supply Box Loadouts
  // (1 will run normaly, 0 will leave them to vanilla defaults, -1 will clear and leave empty)
  setSupplyBoxLoadouts = 1;
  class SupplyBoxes {
    class ACE_medicalSupplyCrate {
      boxCustomName = "Wild Medical Box";
      TransportItems[] = {
        "ACE_elasticBandage:40",
        "ACE_packingBandage:20",
        "ACE_epinephrine:15",
        "ACE_morphine:15",
        "ACE_adenosine:15",
        "ACE_tourniquet:10",
        "ACE_splint:10",
        "ACE_salineIV:4",
        "ACE_salineIV_500:4",
        "ACE_salineIV_250:10"
      };
    };
    class I_E_CargoNet_01_ammo_F {
       // Add ability to mark the larger supply boxes
       // 0 for no actions
       // 1 for 1x smoke and 1x glow stick
       // 2 for 1x smoke and 1x glow stick, and a quick map marker based off of box name
      addMarkingActions = 2;
      // number of boxes that can be put in the crate, default is four,
      // but should be more than the number of sub boxes desired
      boxCustomName = "Multi Box Drifting";
      boxSpace = 4;
      class Box_NATO_Ammo_F { // doesn't affect configuration (or lack thereof) for this box elsewhere
        boxCount = 2; // default is 1 box
        boxCustomName = "Resupply Box";
        TransportWeapons[] = {"gm_m72a3_oli:2"};
        TransportMagazines[] = {
          "CUP_30Rnd_556x45_Stanag:30",
          "CUP_30Rnd_556x45_Stanag_Tracer_Red:6",
          "CUP_100Rnd_TE4_LRT4_Red_Tracer_762x51_Belt_M:8",
          "CUP_1Rnd_HE_M203:6",
          "CUP_1Rnd_HEDP_M203:4",
          "HandGrenade:8",
          "SmokeShell:5",
          "SmokeShellGreen:1",
          "gm_1Rnd_66mm_heat_m72a3:2"
        };
        TransportItems[] = {
          "ACE_elasticBandage:25",
          "ACE_packingBandage:15",
          "ACE_tourniquet:5",
          "ACE_splint:5"
        };
      };
      // inherit the normal boxes
      class ACE_medicalSupplyCrate: ACE_medicalSupplyCrate {
        boxCustomName = "Tame Medical Box";
      };
    };
  };
};

TODO

  • 4x ACE Cargo Slots (customizable)
  • Can be carried by a human
  • Takes up minimal Cargo slots in a truck (1-2 ACE Cargo - aka yes, you can fit more on the inside. Idk how to deal with that yet)
  • Can be ACE interacted with to open a submenu "Marking" > options to activate a chemlight, flare, smoke, and/or mark itself on the map with a label

@lambdatiger lambdatiger added the WIP label Sep 8, 2024
Copy link
Contributor

@TheCandianVendingMachine TheCandianVendingMachine left a comment

Choose a reason for hiding this comment

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

small fixes for consistency

@lambdatiger
Copy link
Contributor Author

The requires an ACE version after 2024/08/27 or the ability to pickup and move the larger box of boxes won't propagate across the network.

@lambdatiger lambdatiger removed the WIP label Sep 10, 2024
@TheCandianVendingMachine
Copy link
Contributor

we usually pull master iirc so if it’s in that we are g2g

@lambdatiger
Copy link
Contributor Author

Figured, just didn't want this to get merged and pushed if Potato was updated but not ACE BW

@kil0byt3
Copy link
Member

good clarification for my sake because pabst usually does the ACE BW updates and I do POTATO, good to know so we can sync up

Copy link
Contributor

@TheCandianVendingMachine TheCandianVendingMachine left a comment

Choose a reason for hiding this comment

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

markers need to be side-specific to the unit marking. blufor and opfor marking the same crate should make it visible to blufor and opfor, but their state is not synchronized to each other

edit: this may be the case, i’m just posting this for PR history

@lambdatiger
Copy link
Contributor Author

lambdatiger commented Sep 10, 2024

markers need to be side-specific to the unit marking. blufor and opfor marking the same crate should make it visible to blufor and opfor, but their state is not synchronized to each other

edit: this may be the case, i’m just posting this for PR history

It sure is the case, 1 is the default side channel id and I didn't see Potato changing that.

Copy link
Contributor

@TheCandianVendingMachine TheCandianVendingMachine left a comment

Choose a reason for hiding this comment

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

actually g2g this approval now lol

@PabstMirror PabstMirror added this to the v1.3.45 milestone Sep 11, 2024
@PabstMirror PabstMirror merged commit dae1509 into BourbonWarfare:master Sep 20, 2024
1 check passed
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