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

Bingus Lowercase Fix #2564

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

Conversation

Bomjojuk
Copy link

Описание PR

Легендарнейший фикс имени бингуса с НИЖНЕГО регистра на ВЕРХНИЙ. Слава проклятью 220

Как нужно помечать чендж в локали не ебу, так что если сделал не так - тыкайте

Медиа

Проверки

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Я ознакомился с наставлениями по работе с репозиторием и следовал им при создании PR'а.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения

@github-actions github-actions bot added the Changes: Localization Изменение затронуло файлы ".ftl" label Feb 15, 2025
@Bomjojuk Bomjojuk requested a review from UrPrice February 15, 2025 13:55
@Tris0r
Copy link

Tris0r commented Feb 15, 2025

Он не знает как помечать чейндж в локали...
УВОЛЕН.

@Alwayswannahunt
Copy link

Он не знает как помечать чейндж в локали... УВОЛЕН.

"This comment was marked as abuse"

@Tris0r
Copy link

Tris0r commented Feb 15, 2025

Он не знает как помечать чейндж в локали... УВОЛЕН.

"This comment was marked as abuse"

b861979812a203bbe2f1d7e84060a52b.mp4

@UrPrice
Copy link
Collaborator

UrPrice commented Feb 15, 2025

А по стандарту же ,name с маленькой, и при осмотре ентити, он всегда в UpperCase ставит имя. А шо тут, почему не так?

@Bomjojuk
Copy link
Author

А по стандарту же ,name с маленькой, и при осмотре ентити, он всегда в UpperCase ставит имя. А шо тут, почему не так?

Хз, но в локали у ВСЕХ петов имя в верхнем регистре, а бингус один был таким уродом

@UrPrice
Copy link
Collaborator

UrPrice commented Feb 16, 2025

А по стандарту же ,name с маленькой, и при осмотре ентити, он всегда в UpperCase ставит имя. А шо тут, почему не так?

Хз, но в локали у ВСЕХ петов имя в верхнем регистре, а бингус один был таким уродом

Ну это ворос к тому, что значит у неписей не совсем правильно отрабатывают наименования. Это можем пропустить конечно как костыль, но давай мы либо в рамках этого PR пофиксим нейминги не на уровне того, как они предоставляются в FTL, а на уровне того, как этот FTL обрабатывается для таких неписей, либо просто заведи на основе этого ишуе, что FTL неймингов для неписей не переходит в игру с заглавной буквой. Выбирай сам

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменение затронуло файлы ".ftl"
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants