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

Translate user-facing messages from Russian to English for better reach #14

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

T4ras123
Copy link

Translated everything to english to allow non-russian speaking users to deploy this template

Copy link
Author

@T4ras123 T4ras123 left a comment

Choose a reason for hiding this comment

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

LGTM

@NikolaySimakov
Copy link
Owner

Okay, but try to put all text messages into a separate json file (preferably several, at i18n/en.json, i18n/ru.json).

Write a class to extract messages from .json files and based on the language that will be written by the developer in .env in the BOT_LANGUAGE variable. You can cache the value of a language variable using lru_cache, for example (just remove the time limit).

Also add documentation to the README for properties (add an example using BOT_LANGUAGE, it's easy).

@T4ras123
Copy link
Author

Alright, imma do that. Give me some time

@NikolaySimakov
Copy link
Owner

Good, I'll look at everything soon and accept it.

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