feat(multi-language): support log translation (DE, EN)

[noppelmax]

As some of my colleges are having trouble with the german-only interface, I decided to translate it to english. I used python-i18n with YAML files therefore.

Regards,
Max

[noppelmax]

Did I use master?! Sorry for that! That happens to me all the time!

Note: I'm no native english speaker. ;) So feel free to fix/optimize the translations.

[haslersn]

I don't see any language files in the PR. Did you forget to add them?

[noppelmax]

Sry missed them

[haslersn]

Ich sehe bei diesem PR die Gefahr, dass er für die Maintainer den Wartungsaufwand unverhältnismäßig erhöht.

Andererseits finde ich dieses Feature sinnvoller als die GUI.

[timreibe]

Did I use master?! Sorry for that! That happens to me all the time!

macht nix, ich merge die PR's ab und zu in den Master anstatt in die Beta, deshalb versuche ich da immer direkt am Anfang drauf zu achten haha

edit: sorry, wollte nicht closen.

[timreibe]

kann man irgendwie die Sprache einstellen?

[noppelmax]

Ich sehe bei diesem PR die Gefahr, dass er für die Maintainer den Wartungsaufwand unverhältnismäßig erhöht.

Andererseits finde ich dieses Feature sinnvoller als die GUI.

Jep. Die Gefahr besteht.

[noppelmax]

kann man irgendwie die Sprache einstellen?

Der sollte automatisch die Systemsprache verwenden. Fallback is de

[noppelmax]

Aber ich hab auch über ein argument nachgedacht: --lang=de

[haslersn]

Aber ich hab auch über ein argument nachgedacht: --lang=de

Systemsprache ist erst mal ok.

[haslersn]

Ich bekomme beim Ausführen:

Traceback (most recent call last):
  File "./main.py", line 530, in <module>
    i18n.load_path.append(os.path.join(PATH, "i18n"))
AttributeError: module 'i18n' has no attribute 'load_path'

[noppelmax]

Ich finde das Feature genial - allerdings würde ich vorschlagen, dass man es selbst aktiviert mit einem argument wie vorgeschlagen und auch nur in der CLI, nicht im GUI. Es betrifft nur einen kleinen Nutzerkreis und wir sollten das System nach wie vor vom Wartungsaufwand möglichst simpel halten.

Mit GUI hab ich nichts am Hut. :)

[noppelmax]

@haslersn Hat es mit den neuen Requirements geklappt? Oder immer noch load_path Probleme? Hab ein neues venv gemacht. requirements.txt ist vollständig.

[timreibe]

Bei mir ist alles auf Englisch, weil in den i18n-config "locale": "en" eingetragen ist. Wenn ich das manuell auf de ändere, sind die Anzeigen wieder auf deutsch.

[timreibe]

eah, moment ich probiere es mit deinem commit

edit: klappt, super!

[noppelmax]

Haben wir eine Möglichkeit alle Pfad mal durch zu rattern? Gibt es Unittest o.ä.? Nicht dass ich was kaputt gemacht hab...

[timreibe]

Haben wir eine Möglichkeit alle Pfad mal durch zu rattern? Gibt es Unittest o.ä.? Nicht dass ich was kaputt gemacht hab...

öh, nein gibts nicht haha. müssen wir so testen

[noppelmax]

Gibt es eine Beschreibung der API? Dann könnte ich evlt. bis Sonntagabend Unittest schreiben.

[timreibe]

Gibt es eine Beschreibung der API?

was genau meinst du damit?

[haslersn]

@haslersn Hat es mit den neuen Requirements geklappt? Oder immer noch load_path Probleme? Hab ein neues venv gemacht. requirements.txt ist vollständig.

Ja, damit kann ich es ausführen. Allerdings läuft es auf deutsch, obwohl meine Systemsprache englisch ist.

Kannst du bitte den folgenden Patch anwenden (git apply datei-mit-dem-folgenden-inhalt) für die Nix-Nutzer:

diff --git a/shell.nix b/shell.nix
index 11e4449..fd142a4 100644
--- a/shell.nix
+++ b/shell.nix
@@ -61,6 +61,16 @@ let
         };
         doCheck = false;
       };
+
+      python-i18n = buildPythonPackage rec {
+        pname = "python-i18n";
+        version = "0.3.9";
+        src = fetchPypi {
+          inherit pname version;
+          sha256 = "1s74f7sgay30kj80pqx9aa74d0slwklfzjynzgmsgwsb6v9g75yz";
+        };
+        doCheck = false;
+      };
     };
   };
 in
@@ -77,6 +87,9 @@ mkShell {
       p.cloudscraper
       p.idna
       p.plyer
+      p.python-i18n
+      p.python-prctl
+      p.pyyaml
       p.selenium
       p.urllib3
     ]))

[haslersn]

Gibt es eine Beschreibung der API?

Ne. Bei der hohen Entwicklungsgeschwindigkeit und der niedrigen Erfahrung der Contributors wäre das auch schwierig durchzusetzen.

Problematisch (im Hinblick auf Tests) kommt hinzu, dass diejenigen Funktionen, die mit dem Impfterminservice kommunizieren, wegen häufig auftretenden 429-Responses recht nichtdeterministisch sind.

[timreibe]

der niedrigen Erfahrung der Contributors

geben unser Bestes 🤡

[noppelmax]

Patch ist drin. Außerdem hab ich ein einfach Testscript geschrieben um zumindest mal zu überprüfen ob in jeder Übersetzung die gleich Keys vorkommen und ob es gleich viele sind. Ordner: test

Falls ihr/wir das wo anders haben wollen einfach Bescheid geben.

Zwecks Unittests; Ok. Aber eine Pfadabdeckung sollte auch ohne API Beschreibung funktionieren. :) Mal sehen wann ich Zeit finde.

[noppelmax]

• --lang=en can now be used to set language to english.
• The language can be set in the advanced settings (via l)
• The currently selected language is highlighted here
• I added a line to README.md to guide english-only user directly to the --lang=en option

[noppelmax]

My next ToDos:

• Translate utils.py
• Translate its.py
• Translate kontaktdaten.py
• Write Unittests