Skip to content

qol: sound and audio playback when fax arrives #6861

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

Merged
merged 3 commits into from
May 22, 2025

Conversation

kukurua
Copy link
Contributor

@kukurua kukurua commented May 21, 2025

Описание

QOL. Добавлен звук уведомления об ответе о факсе. Раньше был только мелкий текст, который очень легко пропустить, но теперь есть и вполне себе слышимый звук! Так же поправил не допереведённые текстовые уведомления о факсе.

Причина создания ПР / Почему это хорошо для игры

QOL

Демонстрация изменений

Теперь при ответе ЦК на факс вы услышите уведомление.

Тесты

Запустил игру с изменениями(удачно);
отправлял ответы на факсы(звук есть);
отправлял админ уведомления(звук есть);
отправлял ответы на факсы, когда душа управляла другой куклой, а не той, что отправляла факсы(звука нет);
отправлял админ уведомления, когда душа управляла другой куклой, а не той, что отправляла факсы(звука нет).

@dageavtobusnick dageavtobusnick changed the title qol/soundaddnew/local: sound and audio playback when fax arrives qol: sound and audio playback when fax arrives May 21, 2025
@dageavtobusnick dageavtobusnick added the Локализация PR содержит текст на русском языке и должен быть проверен Ведущим локализатором. label May 21, 2025
@kukurua
Copy link
Contributor Author

kukurua commented May 22, 2025

Будь ты проклят Пери-Дудос!

@littleboobs
Copy link
Contributor

Я еблан. Апрува.

@kukurua
Copy link
Contributor Author

kukurua commented May 22, 2025

Я еблан. Апрува.

Я действительно накосячил и исправил только сейчас. А по поводу "перевода", то там уже было переведено одно сообщение, которое я и использовал в остальных, но сейчас стоит ваш вариант во всех случаях, да.
Спасибо вам за помощь с локализацией.

@dageavtobusnick dageavtobusnick merged commit 7d883cb into ss220-space:master220 May 22, 2025
15 of 19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Локализация PR содержит текст на русском языке и должен быть проверен Ведущим локализатором.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants