Skip to content

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

@DaniilDonskoy
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

@DaniilDonskoy
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