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

Fix missing gender variable #2722

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

c4llv07e
Copy link
Contributor

Описание PR

Добавил недостающую переменную гендера в сообщение о прибытии.

Первый раз пишу ПР корваксу, так что не уверен, стоит ли оборачивать изменения в // Corvax gender fix. Если нужно, быстренько подправлю.

Почему / Баланс

Мне надоело видеть loc: "ru-RU"/"latejoin-arrival-announcement": "Unknown variable: $gender" каждый раз.

Технические детали

Из моба, в которого только что вселился игрок, достаётся HumanoidAppearanceComponent, который там должен быть всегда, и оттуда получается гендер. Если его там всё же не будет, используется Gender.Epicene (они).

Медиа

Требования

image

Критические изменения

Игроки узнают, кто такие женщины.

Список изменений
no cl no fun

@c4llv07e c4llv07e requested a review from Morb0 as a code owner October 31, 2024 06:11
@github-actions github-actions bot added the Status: Needs Review This PR requires new reviews before it can be merged label Oct 31, 2024
@c4llv07e
Copy link
Contributor Author

Чинит одну из ошибок в #2335

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Needs Review This PR requires new reviews before it can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant