There's just one thing I noticed: Since you seem to be using CAIP as a foundation, you specifically credited DeathByPewPew as its author. I'd appreciate it if you'd credit me as well, as I was the person who started "Combined Arms Integration Project" (which was based on my other mod "Weapon Pack Integration Project") but unfortunately was never fully finished - until a couple of days ago, released under another name, but that's a different story. Therefore I provided DeathByPewPew with my files and they took over.
Hi eyteen, glad to see you published a version of CAIP with Munitions support. I love the Munitions mod :)
I noticed you have a lot of Deleted references in your patch. As a matter of best practice, I would recommend never flagging a record as Deleted as that can cause crashes. If someone else has another mod installed that attempts to reference those specific Combined Arms records, they'll immediately CTD. You can instead mark the records as Initially Disabled, which you can see examples of in CAIP for things like placed ammo instances in cells.
Ah, another useful information to take when making mods. Thank you for the critic, sir, it will help a lot for the future of this series (and user convenience too)!
10 comments
There's just one thing I noticed: Since you seem to be using CAIP as a foundation, you specifically credited DeathByPewPew as its author. I'd appreciate it if you'd credit me as well, as I was the person who started "Combined Arms Integration Project" (which was based on my other mod "Weapon Pack Integration Project") but unfortunately was never fully finished - until a couple of days ago, released under another name, but that's a different story. Therefore I provided DeathByPewPew with my files and they took over.
also what should i choose here?
2. Normal version if you don't have Tactical Reload, the other one if you do
I noticed you have a lot of Deleted references in your patch. As a matter of best practice, I would recommend never flagging a record as Deleted as that can cause crashes. If someone else has another mod installed that attempts to reference those specific Combined Arms records, they'll immediately CTD. You can instead mark the records as Initially Disabled, which you can see examples of in CAIP for things like placed ammo instances in cells.
Cheers!