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

Change tracking implants radio rattle from security to medical #2432

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Radezolid
Copy link
Contributor

About the PR

This PR modifies the tracking implant to send the message to the medical channel instead of the security one.

Why / Balance

Theoretically they are meant for prisoners, but it's mainly used to know when command dies. So this adds a extra layer of communication that delays the response from security. This can also be used by an evil paramedic or doctor as an opportunity to do their objectives.

Technical details

YAML change.

Media

image

Requirements

  • I have tested all added content and changes.
  • I have added media to this PR or it does not require an ingame showcase.

Breaking changes

None.

Changelog

🆑

  • tweak: Tracking implants now rattle on the medical channel instead of the security one.

@Radezolid Radezolid requested a review from a team as a code owner December 13, 2024 02:49
@github-actions github-actions bot added S: Needs Review Changes: YML Changes any yml files size/XS Under 16 lines and removed S: Needs Review labels Dec 13, 2024
@FieldCommand
Copy link
Contributor

Why not both channels? Commands usually don't die out of nowhere, usually someone made them dead, so having security knowing too would be good. But letting medical know is also good, as paramedics can get there to assist and/or medical prepare for a high-value patient arriving to medical.

@Radezolid
Copy link
Contributor Author

Radezolid commented Dec 13, 2024

Two reasons:

  1. RadioSystem only allows to use SendRadioMessage on a single channel, so a rattle with multiple channels is not possible and i can't change that due to my smooth brain on C#.
  2. This is another discouragement for sec to stop powergaming trackers, they are meant for prisoners and you already have the AI on crew monitor + paramedic + corpsmen so you don't really need it and removing the trackers is a bad idea.

@Radezolid Radezolid marked this pull request as draft December 14, 2024 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: YML Changes any yml files size/XS Under 16 lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants