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

Requests Console department Fix #1467

Merged

Conversation

m-dzianishchyts
Copy link
Collaborator

@m-dzianishchyts m-dzianishchyts commented Sep 3, 2024

Что этот PR делает

Устанавливает в недостающие зоны имя, которое должно присваиваться консолям запросов при инициализации.
Меняет путь у некоторых зон на дельте для консистентности.
Добавляет юнит тест на случай, если что-то пропущено.

Почему это хорошо для игры

Починились вторички, сообщения приходят на консоли.

Тестирование

Прогнал тесты на картах с мергированием транслейта.
Отправил запрос с мостика в СБ - сообщение отобразилось.
Запросил вторичку в химии меда - пришло сообщение на консоль, с которой запросил, и на мостик.

Changelog

🆑 Maxiemar
fix: Вторичные цели снова можно запросить с помощью консолей запросов.
fix: На консоли запросов снова исправно поступают сообщения.
/:cl:

@github-actions github-actions bot added 🔧 Фикс Переписываем ошибку так, чтобы она проявлялась в других обстоятельствах Керберос Этот ПР изменил что-то на Керберосе (DeltaStation) labels Sep 3, 2024
@m-dzianishchyts
Copy link
Collaborator Author

что ты такое...

Copy link
Collaborator

@dj-34 dj-34 left a comment

Choose a reason for hiding this comment

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

Почему вместо всего этого нельзя просто в ветке translate переделать проверку зон на русские названия?

@dj-34
Copy link
Collaborator

dj-34 commented Sep 3, 2024

увы
image

@m-dzianishchyts
Copy link
Collaborator Author

Почему вместо всего этого нельзя просто в ветке translate переделать проверку зон на русские названия?

Консоли запросов поели говна и используют ckey() для отправки запросов, который просто скипает кириллицу

@m-dzianishchyts
Copy link
Collaborator Author

На эту консоль можно забивать, будет удалена (ParadiseSS13#26675)

image

Copy link
Collaborator

@AyIong AyIong left a comment

Choose a reason for hiding this comment

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

Выглядит всё вроде отлично

@Legendaxe Legendaxe merged commit 3d7f7f9 into ss220club:master Sep 4, 2024
13 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📜 CL валиден 🔧 Фикс Переписываем ошибку так, чтобы она проявлялась в других обстоятельствах Керберос Этот ПР изменил что-то на Керберосе (DeltaStation)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants