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.

Sign up for GitHub

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

Jump to bottom

Languages revive #2628

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

Languages revive #2628

wants to merge 50 commits into from

Conversation

Bomjojuk
Copy link

@Bomjojuk Bomjojuk commented Feb 23, 2025 •
edited by Kirus59
Loading

Описание PR

Ревайв ПРа на языки жопоглазика с пофикшенными конфликтами (надеюсь)

Медиа
image

image
image

Проверки

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • мультиязычность сообщений
  • Сделать более подробные логи использования языков для админов, дополнение команд, фикс серверных логов
  • фиксированная шифровка слов в течении раунда, фиксированная шифровка слова
  • Разделение функционала на слушающего и говорящего
  • переключение языков через ui
  • Фикс возможности выбора доп языка через черты персонажа (сейчас сервер крашится при попытке)
    Прото-задачи:
  • оптимизация бинарного языка для корректной озвучки ттсом
  • Создание кодового "языка" Синдиката
  • Сделать ключи шифрования более удобными для использования посреди предложения

Изменения

🆑 Kirus59, Jopaglazik, Bomjojuk
add: Добавлена система языков!
add: Добавлена кнопка переключения языка по умолчанию (UI снизу справа в чате)
add: Добавлена возможность многоязычного сообщения при использовании специальных тегов.
add: В чертах персонажа, добавлен выбор дополнительного языка.

@github-actions github-actions bot added Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Changes: C# Изменение затронуло файлы ".cs" Merge Conflict Some content has conflicts. labels Feb 23, 2025
@github-actions GitHub Actions
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the Merge Conflict Some content has conflicts. label Feb 23, 2025
@UrPrice
Copy link
Collaborator

UrPrice commented Feb 24, 2025

S C R A M B L E D

@Tris0r
Copy link

Tris0r commented Feb 24, 2025

Бомжжжжук трогает шарпы, вы только поглядите!

@Bomjojuk
Copy link
Author

Bomjojuk commented Feb 24, 2025 •
edited
Loading

Бомжжжжук трогает шарпы, вы только поглядите!

Каво, я только конфликты пофиксил

Kirus59
Kirus59 requested changes Feb 24, 2025
View reviewed changes
Copy link

@Kirus59 Kirus59 left a comment

Choose a reason for hiding this comment

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

ТТС всё еще не работает с языками или как?

@Bomjojuk
Copy link
Author

ТТС всё еще не работает с языками или как?

Апишку вчера не успел пока тыкать, сегодня попробую

@Bomjojuk
Copy link
Author

Bomjojuk commented Feb 24, 2025 •
edited
Loading

@Kirus59 хотел бы сделать помарку, что, пока что я чисто исправил конфликты изначального пра. Касательно правок херни что ты отметил - посмотрю, смогу ли осилить самостоятельно, но пока позже

@Tris0r
Copy link

Tris0r commented Feb 24, 2025

@Kirus59 хотел бы сделать помарку, что, пока что я чисто исправил конфликты изначального пра. Касательно правок херни что ты отметил - посмотрю, смогу ли осилить самостоятельно, но пока позже

не сможешь

@Kirus59
test
804273e
@Kirus59
Revert "test"
961a084 
This reverts commit 804273e.
@Kirus59 Kirus59 marked this pull request as draft February 24, 2025 16:11
@Kirus59 Kirus59 mentioned this pull request Mar 6, 2025
Closed
5 tasks
@Bomjojuk Bomjojuk marked this pull request as ready for review March 8, 2025 21:50
@Bomjojuk
Copy link
Author

Bomjojuk commented Mar 8, 2025

Ебучий рот, мб завтра попробую потыкать чеки

@Bomjojuk Bomjojuk requested review from stalengd and UrPrice March 9, 2025 19:01
@Bomjojuk
Copy link
Author

Bomjojuk commented Mar 9, 2025

Прайса в догонку

@Kirus59 Kirus59 changed the title Languages revive [WiP] Languages revive Mar 10, 2025
@Bomjojuk Bomjojuk changed the title [WiP] Languages revive Languages revive Mar 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Kirus59 Kirus59 Kirus59 requested changes

@UrPrice UrPrice UrPrice approved these changes

@stalengd stalengd Awaiting requested review from stalengd

Requested changes must be addressed to merge this pull request.

Assignees

@stalengd stalengd

@UrPrice UrPrice

Labels
Changes: C# Изменение затронуло файлы ".cs" Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Changes: UI Изменение затронуло файлы ".xaml"
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@Bomjojuk @UrPrice @Tris0r @Kirus59 @stalengd @Jopaglazik