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

Удаление дублей в файлах локализации #39

Merged
merged 11 commits into from
Jun 8, 2024

Conversation

Rinary1
Copy link
Contributor

@Rinary1 Rinary1 commented Jun 7, 2024

About the PR

Кто-то круто накосячил в Locale, очень много строчек имеют копии, непорядок, спамит в консоль при запуске

Why / Balance

Мешает, лучше сейчас чем потом, ибо может быть еще больше стрингов от корвахов

Technical details

Удаляються лишние ftl файлы, копирующиеся стринги

Media

  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Нету?

Changelog

  • fix: Очищены повторяющиеся переводы

@github-actions github-actions bot added the No C# label Jun 7, 2024
@Rinary1 Rinary1 marked this pull request as ready for review June 7, 2024 12:49
@VigersRay
Copy link
Collaborator

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

@Rinary1

This comment was marked as outdated.

@Rinary1
Copy link
Contributor Author

Rinary1 commented Jun 7, 2024

анлак но не все дубли есть у корваха, так-что и на вашей стороне проблема(например биокодированное оружие, cowboy.ftl в акценте, и тд)

@VigersRay VigersRay changed the title Double Locale Strings Fix Удаление дублей в файлах локализации Jun 8, 2024
@VigersRay VigersRay merged commit 5c82638 into space-sunrise:master Jun 8, 2024
7 of 11 checks passed
@Rinary1 Rinary1 deleted the DOUBLE-LANG-FIX branch June 8, 2024 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants