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

Add emote translations to [translate] #953

Merged
merged 23 commits into from
Jan 29, 2024
Merged

Add emote translations to [translate] #953

merged 23 commits into from
Jan 29, 2024

Conversation

m-dzianishchyts
Copy link
Collaborator

@m-dzianishchyts m-dzianishchyts commented Jan 28, 2024 •
edited
Loading

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

Переносим перевод эмоций из модуля в translate. Надо смержить прям вместе с #951. Синхронизация с #938 также рекомендуется.

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

Чистим чистим чистим.

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

Важно верить в хорошее.

image

Changelog

🆑 Maxiemar
tweak: Дальнейший перевод эмоций, включая рукопожатия и пощечины.
/:cl:

@github-actions github-actions bot added the :finnadie: НЕ МОДУЛЬНО Па-ху-ю. label Jan 28, 2024
@m-dzianishchyts m-dzianishchyts marked this pull request as ready for review January 28, 2024 01:57
This was referenced Jan 28, 2024
Merged
Merged
dj-34
dj-34 requested changes Jan 28, 2024
View reviewed changes
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.

Перепроверь muzzled_noises

code/datums/emote.dm Outdated Show resolved Hide resolved
code/datums/status_effects/neutral.dm Outdated Show resolved Hide resolved
code/datums/status_effects/neutral.dm Show resolved Hide resolved
code/datums/status_effects/neutral.dm Show resolved Hide resolved
code/datums/status_effects/neutral.dm Outdated Show resolved Hide resolved
code/modules/mob/living/living_emote.dm Outdated Show resolved Hide resolved
code/modules/mob/living/living_emote.dm Show resolved Hide resolved
code/modules/mob/living/living_emote.dm Outdated Show resolved Hide resolved
code/modules/mob/living/living_emote.dm Show resolved Hide resolved
code/modules/mob/living/living_emote.dm Outdated Show resolved Hide resolved
@dj-34 dj-34 added 📝 Перевод Приятно когда всё на родном языке. and removed :finnadie: НЕ МОДУЛЬНО Па-ху-ю. labels Jan 28, 2024
@m-dzianishchyts @dj-34
Fix typo
c5e0c9c 
Co-authored-by: dj-34 <[email protected]>
@github-actions github-actions bot added the :finnadie: НЕ МОДУЛЬНО Па-ху-ю. label Jan 28, 2024
@m-dzianishchyts
Copy link
Collaborator Author

Перепроверь muzzled_noises

Ну сейчас на ветке так
image

Было так
image

@dj-34
Copy link
Collaborator

dj-34 commented Jan 28, 2024

Перепроверь muzzled_noises

Ну сейчас на ветке так image

Было так image

понял

dj-34
dj-34 approved these changes Jan 28, 2024
View reviewed changes
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.

Должно быть окей

Gaxeer pushed a commit that referenced this pull request Jan 28, 2024
@m-dzianishchyts
Refactor translation of emotes (#938)
2b75cbc 
<!-- Пишите **НИЖЕ** заголовков и **ВЫШЕ** комментариев, иначе что то
может пойти не так. -->
<!-- Вы можете прочитать Contributing.MD, если хотите узнать больше. -->

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

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

![image](https://github.com/ss220club/Paradise-SS220/assets/39908528/46dbe693-e326-4d46-8e46-050317656cb8)
<!-- Опишите, почему, по вашему, следует добавить эти изменения в игру.
-->

## Тестирование
Да.


![image](https://github.com/ss220club/Paradise-SS220/assets/39908528/192ad87d-d9ff-4bb8-b66e-524d9ac8ea91)

![image](https://github.com/ss220club/Paradise-SS220/assets/39908528/e9c3001d-7b92-47e4-9b3d-d7d0e153a8ee)

![image](https://github.com/ss220club/Paradise-SS220/assets/39908528/85e976eb-991f-4075-8eac-2a24ecbd28bd)

![image](https://github.com/ss220club/Paradise-SS220/assets/39908528/c0344bad-f44c-47a7-b5f2-3e75596137e7)
<!-- Как вы тестировали свой PR, если делали это вовсе? -->

## Changelog

:cl: Maxiemar
fix: Некоторые эмоции можно (теперь точно) дополнить, используя их в
Панели эмоций (вкладка IC) и указав цель.
tweak: Переведены все именования эмоций.
/:cl:

<!-- Оба :cl:'а должны быть на месте, что-бы чейнджлог работал! Вы
можете написать свой ник справа от первого :cl:, если хотите. Иначе
будет использован ваш ник на ГитХабе. -->
<!-- Вы можете использовать несколько записей с одинаковым префиксом
(Они используются только для иконки в игре) и удалить ненужные. Помните,
что чейнджлог должен быть понятен обычным игроком. -->
<!-- Если чейнджлог не влияет на игроков(например, это рефактор), вы
можете исключить всю секцию. -->
Legendaxe pushed a commit that referenced this pull request Jan 29, 2024
@m-dzianishchyts
Remove emote translations from [master] (#951)
07665db 
<!-- Пишите **НИЖЕ** заголовков и **ВЫШЕ** комментариев, иначе что то
может пойти не так. -->
<!-- Вы можете прочитать Contributing.MD, если хотите узнать больше. -->

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

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

## Тестирование
Оно реально пропало.
<!-- Как вы тестировали свой PR, если делали это вовсе? -->

## Changelog

:cl: Maxiemar
tweak: Перевод эмоций перенесен из нашего модуля.
/:cl:

<!-- Оба :cl:'а должны быть на месте, что-бы чейнджлог работал! Вы
можете написать свой ник справа от первого :cl:, если хотите. Иначе
будет использован ваш ник на ГитХабе. -->
<!-- Вы можете использовать несколько записей с одинаковым префиксом
(Они используются только для иконки в игре) и удалить ненужные. Помните,
что чейнджлог должен быть понятен обычным игроком. -->
<!-- Если чейнджлог не влияет на игроков(например, это рефактор), вы
можете исключить всю секцию. -->
@Legendaxe Legendaxe merged commit 9b7340c into ss220club:translate Jan 29, 2024
6 checks passed
@m-dzianishchyts m-dzianishchyts deleted the add-emote-translations branch January 29, 2024 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dj-34 dj-34 dj-34 approved these changes

Assignees
No one assigned
Labels
:finnadie: НЕ МОДУЛЬНО Па-ху-ю. 📝 Перевод Приятно когда всё на родном языке.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@m-dzianishchyts @dj-34 @Legendaxe