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

Небольшая проблема перевода снижения кода до красного #689

Closed
1 task done
MrRomainzZ opened this issue Nov 13, 2023 · 3 comments

Comments

@MrRomainzZ
Copy link

Exploit Reports

  • I confirm this issue is not an exploit. (Required)

BYOND Version

515.1620

Issue Description

При понижении кода с гаммы до красного пишется "Механзим самоуничтожения станции деактивирован", когда механизм самоуничтожения запускается с дельты и выше

What did you expect to happen?

Оповещение, не вводящее в заблуждение

What happened instead?

Меня обманули :(

Why is this bad/What are the consequences?

Думаю, некоторых это может путать

Steps to reproduce the issue.

Снизьте код с любого выше красного до красного

When did the problem start happening?

Там вроде анонсы переводили. Думаю, с оного #672

Extra information

изображение

Relevant log output/runtime error

No response

@Gaxeer
Copy link
Collaborator

Gaxeer commented Nov 13, 2023

Текущая система кодов не позволяет отслеживать с какого конкретно кода было понижено, чтобы делать какие-то адаптивные сообщения

@MrRomainzZ
Copy link
Author

Будет пофикшено ПР-ом #693

@AyIong
Copy link
Collaborator

AyIong commented Nov 13, 2023

Мог просто ПР привязать

AyIong pushed a commit that referenced this issue Nov 13, 2023
<!-- Пишите **НИЖЕ** заголовков и **ВЫШЕ** комментариев, иначе что то
может пойти не так. -->
<!-- Вы можете прочитать Contributing.MD, если хотите узнать больше. -->

## Что этот PR делает
Убирает текст "Механзим самоуничтожения станции деактивирован" при
понижении кода до красного слегка заменяя его

<!-- Вкратце опишите изменения, которые вносите. -->
<!-- Опишите **все** изменения, так как противное может сказаться на
рассмотрении этого PR'а! -->
<!-- Если вы исправляете Issue, добавьте "Fixes #1234" (где 1234 - номер
Issue) где-нибудь в описании PR'а. Это автоматически закроет Issue после
принятия PR'а. -->

## Почему это хорошо для игры
Меньше путаницы

<!-- Опишите, почему, по вашему, следует добавить эти изменения в игру.
-->

<!-- Если вы не меняли карту или спрайты, можете опустить эту секцию.
Если хотите, можете вставить видео. -->

<!-- Как вы тестировали свой PR, если делали это вовсе? -->

## Changelog

:cl: Romains
fix: Убрал текст "Механзим самоуничтожения станции деактивирован" при
понижении уровня угрозы до красного кода
/:cl:

<!-- Оба :cl:'а должны быть на месте, что-бы чейнджлог работал! Вы
можете написать свой ник справа от первого :cl:, если хотите. Иначе
будет использован ваш ник на ГитХабе. -->
<!-- Вы можете использовать несколько записей с одинаковым префиксом
(Они используются только для иконки в игре) и удалить ненужные. Помните,
что чейнджлог должен быть понятен обычным игроком. -->
<!-- Если чейнджлог не влияет на игроков(например, это рефактор), вы
можете исключить всю секцию. -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants