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.

Sign up for GitHub

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

Jump to bottom

ACE/KAM Painkillers config conflict #546

Closed
AgentDove opened this issue Jun 7, 2024 · 1 comment · Fixed by #548
Closed

ACE/KAM Painkillers config conflict #546

AgentDove opened this issue Jun 7, 2024 · 1 comment · Fixed by #548

Comments

@AgentDove
Copy link

Description:
Ace recently added their own painkillers, and its uses the config name 'PainKillers' in the Ace Medical Treatment cfg's.
Kat also uses the same config name for its painkillers.

Steps to reproduce:

  1. Load into Ace virtual arsenal.
  2. Add "ace_painkillers" to your inventory
  3. Try to use said painkillers on your head.
  4. No option appears.

Where did the issue occur?

  • Self-Hosted Multiplayer / Virtual Arsenal

Additional information:
ACE_Medical_Treatment_Actions
Ace Painkillers - Config
KAM Painkillers - Config
ACE_Medical_Treatment >> Medication
Ace Painkillers - Config
KAM Painkillers - Config

The items do not conflict with each other, only the medical configs.
@Cplhardcore
Copy link
Contributor

Just a note, if you are running KAM, you should also be running the painkillers as well, as they do more then just remove pain

@Apricot-ale Apricot-ale mentioned this issue Jun 7, 2024
Merged
MiszczuZPolski added a commit that referenced this issue Jun 26, 2024
@Apricot-ale @MiszczuZPolski
Pharma - Change KAM Painkillers to CWMP (#548)
e8d0684 
**When merged this pull request will:**
- _Fix #546 problem._
- _Rename whole of (KAT) Painkillers thing (except item class name) to
CWMP (from [TCCC
14.3](https://tccc.org.ua/en/guide/combat-wound-medication-pill-pack))
and item stringtable to Combat-Pill-Pack (from [DD 1380 TCCC CASUALTY
CARD](https://learning-media.allogy.com/api/v1/pdf/eb6c36b5-edbf-4e68-ae47-7a6e6536c6e7/contents))_
- _Max "ammo" count is now 10->4, weight is 1->0.5 (Guide need to
change)_

New icon;

![icon_painkillers](https://github.com/KAT-Advanced-Medical/KAM/assets/50947830/41a0e77c-ce68-4724-8526-ec888f114802)
New icon with higher resolutions (you know where it come from...)
when 64x64, almost text are can't read and texture not detailed so it's
will be ok

![icon_painkillers_256](https://github.com/KAT-Advanced-Medical/KAM/assets/50947830/81af1dba-0d95-4ed9-a42c-a687f5e8f2c7)



### IMPORTANT

- [Development Guidelines](https://ace3.acemod.org/wiki/development/)
are read, understood and applied.
- Title of this PR uses our standard template `Component -
Add|Fix|Improve|Change|Make|Remove {changes}`.

---------

Co-authored-by: MiszczuZPolski <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Pharma - Change KAM Painkillers to CWMP Apricot-ale/KAM
2 participants
@AgentDove @Cplhardcore