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

No death #741

Merged
merged 17 commits into from
Feb 17, 2024
Merged

No death #741

merged 17 commits into from
Feb 17, 2024

Conversation

ZoNeSRuS
Copy link

Описание PR

Долгожданный внес клонерки на помойку(в тир 3 РНД)
Конерка:

  1. Платы удалены со всех карт.
  2. Добавлено в тир 3 исследования РНД.

Возрождение:
Новая фичча позволяющая госту вернуться в лобби. через 20 минут после смерти.

Сдаться в крите:
Удалено у всех кукл игроков. Животные и другие все еще имеют эту кнопку.

Последние слова:
Теперь РЕАЛЬНО последние слова. После их произнесения вы не сможете вернуться в тело.

Медиа

Проверки

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

Изменения

🆑 ZoNeSRuS

  • add: Добавлена кнопка "возрождение"
  • remove: Удалена возможность "сдаться" в крите, исключение животные.
  • tweak: Изменены "последние слова". Теперь после них вы не сможете ожить.
  • tweak: Клонерка удалена и перемещена в РНД.

@ZoNeSRuS
Copy link
Author

Да йопта где ошибки.

@VladIsLove-e
Copy link

Теперь последние слова не юзабельны,
Предлагаю сделать так: с момента Крита можно написать последние слова, но до момента смерти они не отправится. Через условные 5 секунд от крита кнопка убирается. Если человек был в Крите дольше(к примеру)10 секунд, последний слова произносятся шёпотом (так их и издалека не слышно и это логичнее, мол ослабел)

@ZoNeSRuS
Copy link
Author

Теперь последние слова не юзабельны, Предлагаю сделать так: с момента Крита можно написать последние слова, но до момента смерти они не отправится. Через условные 5 секунд от крита кнопка убирается. Если человек был в Крите дольше(к примеру)10 секунд, последний слова произносятся шёпотом (так их и издалека не слышно и это логичнее, мол ослабел)

С радостью бы выслушал эти слова на собрании разрабов, где все это обсуждалось и утверждалось, но ты не пришел.

@VladIsLove-e
Copy link

Теперь последние слова не юзабельны, Предлагаю сделать так: с момента Крита можно написать последние слова, но до момента смерти они не отправится. Через условные 5 секунд от крита кнопка убирается. Если человек был в Крите дольше(к примеру)10 секунд, последний слова произносятся шёпотом (так их и издалека не слышно и это логичнее, мол ослабел)

С радостью бы выслушал эти слова на собрании разрабов, где все это обсуждалось и утверждалось, но ты не пришел.

Злюка

@github-actions github-actions bot added the Changes: Localization Изменение затронуло файлы ".ftl" label Feb 16, 2024
Comment on lines 19 to 20
Gold: 100 No newline at end of file
Gold: 100

Choose a reason for hiding this comment

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

???

Copy link
Author

Choose a reason for hiding this comment

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

работал с этим файлом, потом изменения откатил.

Comment on lines 121 to 129
//SS-220
private void OnActionRespanw(EntityUid uid, GhostComponent component, RespawnActionEvent args)
{
if (!TryComp<ActorComponent>(uid, out var actor))
return;

_host.ExecuteCommand(actor.PlayerSession, "respawn");
}
//SS-220 end

Choose a reason for hiding this comment

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

во всех комментах с пометками нашего кода желательно добавить и название фичи/ветки, на которой производились изменения

//SS220-respawn begin
DoStuff();
//SS220-respawn end

типа такого

@@ -454,6 +454,7 @@
- ArtifactCrusherMachineCircuitboard
- TelecomServerCircuitboard
- MassMediaCircuitboard

Choose a reason for hiding this comment

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

???

Copy link
Author

Choose a reason for hiding this comment

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

работал с этим файлом, потом изменения откатил.(2)

Choose a reason for hiding this comment

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

тут также надо бы изменения убрать

Copy link
Author

Choose a reason for hiding this comment

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

тут также надо бы изменения убрать

Да понял, тут рил мой косяк.

@@ -17,4 +17,4 @@
Steel: 100
Glass: 900
Silver: 100
Gold: 100

Choose a reason for hiding this comment

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

все еще здесь прикол

Copy link
Author

Choose a reason for hiding this comment

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

все еще здесь прикол

Тут убрался символ табуляции после 100. Я ХЗ как его назад вернуть.

Content.Server/Ghost/GhostSystem.cs Outdated Show resolved Hide resolved
@@ -454,6 +454,7 @@
- ArtifactCrusherMachineCircuitboard
- TelecomServerCircuitboard
- MassMediaCircuitboard

Choose a reason for hiding this comment

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

тут также надо бы изменения убрать

@ZoNeSRuS ZoNeSRuS requested a review from DexlerXD February 17, 2024 11:32
@DexlerXD DexlerXD merged commit 6005f8e into SerbiaStrong-220:master Feb 17, 2024
12 checks passed
@ZoNeSRuS ZoNeSRuS deleted the noDeath branch February 22, 2024 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменение затронуло файлы ".ftl" Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants