UI and i18n Updates #2436
Draft
UI and i18n Updates #2436
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Adds setalias command. Adds several dev-only commands for project maintenance. Adds option for Opus audio. Adds option to remove embed footer. Adds sub-command syntax for permissions. Adds optional new permissions management. Enables MusicBot to generate example config & perms files. Removes cruft .travis.yml file. Rename oauth2 username to avoid conflict with yt-dlp. Replace docstring help with `command_helper` decorator. Replace JSON i18n with gettext, with support for logs and discord. Extend python logger class rather than virtually construct it. (this might fix 3.13) Touch nearly every printed string in MusicBot. :)
…ist. Proper fix for generating missing options and permissions files. Enable moving all writable directories with --write-dir cli flag. Convert some more constants for the above cli option.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
devbranch (the latest developmental version), and this pull request is targeting that branch as a baseDescription
The massive UI / i18n PR is here. As usual, there was some scope creep and this does need extra testing.
The short list of changes is thus:
--write-dirCLI flag..travis.ymlfile.@command_helperdecorator.Responseobjects fromdiscord.Embedto make UseEmbeds option consistent.Please see the ./i18n/readme.md file for more info on how translations work and how to use them.
While this PR removes all the existing
./config/i18n/files, I have attempted to replace some of the existing translations using Machine Translation from Crowdin primarily.I'm sure the translations could use some work. I left the
crowdin.ymlfile in this PR as a template for setting up crowdin on the main repository (or forks), following this PR. I think it is a pretty good way to keep track of translations personally and would suggest it be set up on the main repo if maintainers are willing to allow/use it.I hope these changes strike you as sensible and acceptable.