ACE3
ACE3 copied to clipboard
acemod
Medical Treatment - Add Additional Conditions for PAK Use
When merged this pull request will: Personal Aid Kits now have additional optional conditions for use:
- Minimum Blood Level
- Minimum Heart Rate
- Patient must not be in pain
- Patient must have no open wounds
- Patient must have no open or bandaged wounds (Must be fully stitched)
This PR is an attempt to resuscitate the work done in stale PR #7684, which was created prior to major build pipeline changes. The actual feature work has been recreated and re-tested here.
IMPORTANT
- [x] If the contribution affects the documentation, please include your changes in this pull request so the documentation will appear on the website.
- [x] Development Guidelines are read, understood and applied.
- [x] Title of this PR uses our standard template
Component - Add|Fix|Improve|Change|Make|Remove {changes}.
It seems reasonable to give mission makers more control over the "embedded" aspects of ACE's medical system. Without PRs like these, communities have to resort to making in-house mods to expose the underlying features as CBA options. Which is fine if you only have to do this for one mod, but repeat that process a few times and your addon dependency graph quickly becomes more intricate than the lineage of the British royal family.
Jokes aside - I appreciate your dedication for keeping this submission going, and hope this gets more attention this time! 🤞
As a missionmaker I would very much appreciate this being added, since currently I don't give out PAKs since it represents an "auto-heal" button in an otherwise pretty realistic representation of how med-care should work. Tying the PAK to a condition would allow to be an actual tool that rewards proper care by lower-tier medics until the pt can be seen ba a doc.
@rautamiekka Just bumping this as it seems to be falling under the radar. Thanks for any context you can provide.
@veteran29 Apologies for the ping, but you have been active in the repository recently and am trying to get movement on this PR. These changes have been approved in the past and gone through several rounds of feedback from @mharis001, but it seems to be falling under the radar since.
Thanks!
Kindly, after nearly 3 years of trying to get this merged, it is no longer worth redeveloping this yet again versus maintaining an override in our framework.