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 ace medical painkillers #156

Closed
wants to merge 2 commits into from
Closed

add ace medical painkillers #156

wants to merge 2 commits into from

Conversation

Drofseh
Copy link
Contributor

@Drofseh Drofseh commented Nov 29, 2021

  • add painkillers to ace medical
  • reduce pain, but only 1/8th as much as morphine
  • safer to use, doesn't reduce HR, but can still OD if too many are taken too quickly (25 doses within 20 minutes)
  • can only be used when conscious

- add painkillers to ace medical
- reduce pain, but only 1/8th as much as morphine
- safer to use
- can only be used when conscious
Copy link
Member

@Freddo3000 Freddo3000 left a comment

Choose a reason for hiding this comment

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

tbh this should probably be a PR to ACE, not here.

addons/ace_tweaks/CfgWeapons.hpp Outdated Show resolved Hide resolved
addons/ace_tweaks/CfgWeapons.hpp Outdated Show resolved Hide resolved
addons/ace_tweaks/CfgWeapons.hpp Outdated Show resolved Hide resolved
addons/ace_tweaks/stringtable.xml Outdated Show resolved Hide resolved
addons/ace_tweaks/CfgWeapons.hpp Outdated Show resolved Hide resolved
addons/ace_tweaks/CfgVehicles.hpp Outdated Show resolved Hide resolved
addons/ace_tweaks/ACE_Medical_Treatment_Actions.hpp Outdated Show resolved Hide resolved
addons/ace_tweaks/ACE_Medical_Treatment_Actions.hpp Outdated Show resolved Hide resolved
addons/ace_tweaks/ACE_Medical_Treatment.hpp Show resolved Hide resolved
@Freddo3000 Freddo3000 added the enhancement New feature or request label Nov 29, 2021
@Drofseh
Copy link
Contributor Author

Drofseh commented Nov 29, 2021

tbh this should probably be a PR to ACE, not here.

This is actually partially implemented in Ace already, but it was never finished for reasons I don't know.
Will implement suggestions.

@BorderKeeper
Copy link
Contributor

I think it might be neat for our community to include these unique features that other groups dont have.

@Freddo3000
Copy link
Member

I think it might be neat for our community to include these unique features that other groups dont have.

It might be neat, though I don't think it is healthy for the modding community. Think of how many features we'd loose out on if all communities resonated that way?

@BorderKeeper
Copy link
Contributor

I think it might be neat for our community to include these unique features that other groups dont have.

It might be neat, though I don't think it is healthy for the modding community. Think of how many features we'd loose out on if all communities resonated that way?

You can see it as us being the testbed needed for the implementation to make it into ACE. We are not usurping features but testing them so it can be obvious that they are a good addition.

@BorderKeeper
Copy link
Contributor

BorderKeeper commented Dec 2, 2021

After discussion with @Freddo3000. Please make a #suggestion post about this. If it gets positive feedback we can merge this PR.

Copy link
Contributor

@BorderKeeper BorderKeeper left a comment

Choose a reason for hiding this comment

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

Tally from suggestion post on 2nd of December is 9 yes to 1 no. Approved. (https://discord.com/channels/240160552867987475/374416645969543178/916021343567364197)

Copy link
Contributor

@BorderKeeper BorderKeeper left a comment

Choose a reason for hiding this comment

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

lgtm

why did I approve again

@Drofseh Drofseh requested a review from Freddo3000 May 9, 2022 02:48
@Freddo3000
Copy link
Member

Decided against implementing on a staff meeting, see https://discord.com/channels/240160552867987475/374416645969543178/916021343567364197

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants