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

Make chat & betterfps warnings translatable #377

Merged
merged 3 commits into from
Nov 6, 2023

Conversation

kotmatross28729
Copy link

This is the only pull request, where I don't ask a question.

Copy link
Owner

@Roadhog360 Roadhog360 left a comment

Choose a reason for hiding this comment

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

Things highlighted in review should be changed before this PR is accepted
(forgor to do this earlier lol)

efr.prerelease.version=§c§l[Debug]: §rYou are using a pre-release version of §bEt §bFuturum §bRequiem§r. This version might not be stable, click here to go to GitHub to report bugs.
efr.WARN.prerelease.version=WARNING: a pre-release version of Et Futurum Requiem is in use! This build may not be stable, expect more bugs than usual.
efr.WARN.prerelease.version.bugs=Be sure to report bugs at https://github.com/Roadhog360/Et-Futurum-Requiem/issues
Copy link
Owner

Choose a reason for hiding this comment

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

Capitalization inconsistency, all keys should follow a lower_snake_case format between each dot


efr.prerelease.version=§c§l[Debug]: §rYou are using a pre-release version of §bEt §bFuturum §bRequiem§r. This version might not be stable, click here to go to GitHub to report bugs.
efr.WARN.prerelease.version=WARNING: a pre-release version of Et Futurum Requiem is in use! This build may not be stable, expect more bugs than usual.
Copy link
Owner

Choose a reason for hiding this comment

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

This key should contain "log", not "WARN",

efr.decoration.only=Decoration only

efr.prerelease.version=§c§l[Debug]: §rYou are using a pre-release version of §bEt §bFuturum §bRequiem§r. This version might not be stable, click here to go to GitHub to report bugs.
Copy link
Owner

Choose a reason for hiding this comment

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

Does not describe that it's a warning. Should get "warn" instead of the below key

efr.WARN.prerelease.version=ВНИМАНИЕ: используется предварительная версия Et Futurum Requiem! Эта сборка может быть нестабильной, ожидайте больше ошибок, чем обычно.
efr.WARN.prerelease.version.bugs=Обязательно сообщайте об ошибках по адресу https://github.com/Roadhog360/Et-Futurum-Requiem/issues.
efr.CRITICAL.badBetterFPSAlgorithm1=§4§l[КРИТИЧНО]: §bBetterFPS §rнастроен с плохим алгоритмом.
efr.CRITICAL.badBetterFPSAlgorithm2= "rivens-half" или "taylors" не должны использоваться для предотвращения ущерба вашему миру. Настоятельно рекомендуется зайти в его конфигурацию и изменить "алгоритм" на что-нибудь другое, желательно "vanilla".
Copy link
Owner

Choose a reason for hiding this comment

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

"алгоритм" in quotes should be "algorithm", config values are not translated, there is only a "algorithm" option, no "алгоритм" option

@kotmatross28729
Copy link
Author

@Roadhog360, ?

@Roadhog360
Copy link
Owner

Roadhog360 commented Nov 5, 2023

BetterFPS keys still contain capital letters

@kotmatross28729
Copy link
Author

@Roadhog360

@Roadhog360 Roadhog360 merged commit ccc200e into Roadhog360:master Nov 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants