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

Ninja targets update #2148

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

Alwayswannahunt
Copy link

@Alwayswannahunt Alwayswannahunt commented Oct 24, 2024

Описание PR

дд - https://discord.com/channels/1158128815311171638/1290759428634574931/1297207471030534226

Цели
Ниндзя получил три новые цели из двух новых типов целей - "Заказ на устрашение" и "Подставить сотрудника".
Убраны цели на взрыв паучьей бомбы и взлом консоли криминальных записей.

Инструменты
Маска ниндзя теперь умеет изменять ТТС и голос носителя.

Код
Добавлены ивенты при изменении криминальных записей.

Медиа
TBA

Проверки

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Я ознакомился с наставлениями по работе с репозиторием и следовал им при создании PR'а.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения

🆑 AlwyAnri (код), Kemran (спрайты)

  • add: Добавлены новые цели для ниндзя - "Заказ на устрашение" и "Подставить сотрудника"!
  • remove: Убраны цели ниндзя на взрыв бомбы и взлом консоли криминальных записей!
  • tweak: Маска ниндзя теперь имеет свойства голосовой маски!

@TryHardo7
Copy link

Для ниндзи крайне легко выполнить заказ на устранение любого на станции, я считаю не нужна ему такая цель. Подставить - отлично, пакостить - ок, воровать - ок.
Может цель на взлом боргов? Они никогда этого не делают
Какое-то рп типа случайному члену экипажа без МЩ прокает цель стать учеником ниндзи и что-то в этом русле сделать?

@Alwayswannahunt
Copy link
Author

Для ниндзи крайне легко выполнить заказ на устранение любого на станции, я считаю не нужна ему такая цель. Подставить - отлично, пакостить - ок, воровать - ок. Может цель на взлом боргов? Они никогда этого не делают Какое-то рп типа случайному члену экипажа без МЩ прокает цель стать учеником ниндзи и что-то в этом русле сделать?

Заказ на устрашение

Ниндзя работающий за РНД уже и так стал мемом, он теперь будет и боргов ещё собирать, подрабатывая утилём. Не, пожалуй, не этим он должен заниматься.

РП рпом, но цель засчитывать надо, никто не мешает написать в ахелп и отыграть такое с разрешения/помощи админа. Я уже молчу что ниндзя делать если ученик не хочет учиться? Любой ответ на этот вопрос сведёт на ноль интересность цели.

@github-actions github-actions bot added Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Changes: C# Изменение затронуло файлы ".cs" labels Oct 26, 2024
@github-actions github-actions bot added the Changes: Sprites Изменение затронуло файлы ".rsu / .png" label Dec 1, 2024
Copy link

github-actions bot commented Dec 1, 2024

RSI Diff Bot; head commit 44a8d30 merging into 0bd3b2f
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/SS220/Objectives/frame.rsi

State Old New Status
snake_cian Added
snake_green Added
snake_white Added

Resources/Textures/SS220/Objectives/intimidate.rsi

State Old New Status
brute Added
burn Added
toxin Added

Edit: diff updated after 44a8d30

@Alwayswannahunt
Copy link
Author

@stalengd Я открою, но мерджить после тестов

@Alwayswannahunt Alwayswannahunt marked this pull request as ready for review December 6, 2024 20:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Изменение затронуло файлы ".cs" Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Changes: Sprites Изменение затронуло файлы ".rsu / .png"
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants