MyFinances icon indicating copy to clipboard operation
MyFinances copied to clipboard

[NEEDS THOUGHT] added initial translation setup

Open introkun opened this issue 10 months ago • 1 comments

  • added initial translation setup (English, Russian)
  • removed autopep8 and flake8 from pre-commit
  • removed double-quote-string-fixer
  • disabled mypy due to #294

Checklist

  • [x] Ran the Black Formatter and djLint-er on any new code (checks will fail without)
  • [x] Made any changes or additions to the documentation where required
  • [x] Changes generate no new warnings/errors
  • [x] New and existing unit tests pass locally with my changes

What type of PR is this?

  • ✨ Feature
  • ♻️ Code Refactor

Added/updated tests?

  • 🙅 no, because they aren't needed

Related PRs, Issues etc

  • Related Issue #294

introkun avatar Apr 05 '24 00:04 introkun

marked this pull request as draft Any comments @TreyWW ?

introkun avatar Apr 06 '24 18:04 introkun

Hmm I've been looking over this a few times over the last week and I am considering it. It's just I feel like it's going to make all of the messages much harder. Like would formatting (e.g ) work in the messages?

And with the Django tags, trans, anything dynamic is just a pain to put in tags (I believe).

It could be a good option, I'll open up the PR again and maybe give it a go, see how it is.

Sorry about the delay

TreyWW avatar May 29 '24 10:05 TreyWW