Feature/viewer 1344 locale keys bereinigen #1405

nOester · 2024-12-17T09:39:53Z

Hallo @vej-ananas , @daniel-va
Wie verabredet, habe ich eine PR mit meinen Anpassungen für die Aktualisierung der Übersetzungs-Files erstellt.
Danke für's Revieren

…nigen

sonarqubecloud · 2024-12-17T09:40:33Z

Quality Gate passed

Issues
3 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

daniel-va · 2025-01-06T06:40:57Z

ui/i18next-scanner.config.cjs

+<<<<<<< HEAD
+// import path from 'node:path';
+=======
+//import path from 'node:path';
+>>>>>>> test_viewer


Das sind noch Überbleibsel von einem Merge Conflict. Du solltest du ohne Probleme entfernen können.

Suggested change

<<<<<<< HEAD

// import path from 'node:path';

=======

//import path from 'node:path';

>>>>>>> test_viewer

Nach <<<<<< HEAD suchen und löschen

daniel-va · 2025-01-06T06:41:43Z

ui/i18next-scanner.config.cjs

+<<<<<<< HEAD
+      savePath: path.join(__dirname, 'locales/{{lng}}.{{ns}}-test.json'),
+=======
+      savePath: path.join(__dirname, 'locales/empty/{{ns}}-{{lng}}-empty.json'),
+>>>>>>> test_viewer


Dies sind ebenfalls noch Merge Conflicts, die must du wohl manuell überprüfen & anpassen.

Hier prüfen, welche Zeile korrekt ist

daniel-va · 2025-01-06T06:41:52Z

ui/i18next-scanner.config.cjs

+<<<<<<< HEAD
+    removeUnused: true, // Diese Option sorgt dafür, dass unbenutzte Keys entfernt werden
+=======
+    // removeUnused: true, // Diese Option sorgt dafür, dass unbenutzte Keys entfernt werden
+>>>>>>> test_viewer


Dies sind ebenfalls noch Merge Conflicts, die must du wohl manuell überprüfen & anpassen.

Hier prüfen, welche Zeile korrekt ist

daniel-va · 2025-01-06T06:44:31Z

ui/locales/README.MD

+npx i18next-scanner --config i18next-scanner.config.cjs
+```
+
+## 2) Run script to campare and combine translations


Schreibfehler.

Suggested change

## 2) Run script to campare and combine translations

## 2) Run script to compare and combine translations

Schreibfehler lokal korrigieren

daniel-va · 2025-01-06T06:44:45Z

ui/locales/README.MD

+```bash
+cd swissgeol-viewer-app/ui
+```
+- (*optional*) Modifiy configuration:  


Schreibfehler.

Suggested change

- (*optional*) Modifiy configuration:

- (*optional*) Modify configuration:

Schreibfehler lokal korrigieren

daniel-va · 2025-01-06T06:50:29Z

ui/locales/app.de.json

@@ -240,6 +241,11 @@
  "tbx_create_section_label": "Schnitte erstellen",
  "tbx_cross_sections_depth_label": "Schnitttiefe (m ü. M.)",
  "tbx_cross_sections_format_label": "Format des Schnitts",
+<<<<<<< HEAD
+=======
+  "tbx_data-download": "Datendownload",


Sollte dieser Key nicht statt einem Dash eine Underline zwischen data und download nutzen?

Suggested change

"tbx_data-download": "Datendownload",

"tbx_data_download": "Datendownload",

_ in den Übersetzungskey im Code auf - ändern

daniel-va · 2025-01-06T06:52:56Z

ui/locales/origLocals_backUp/app.de.json

@@ -0,0 +1,356 @@
+{


Ich nehme mal an, dass die Dateien origLocals_backUp/*.json den Zustand vor deinen Änderungen darstellen? Falls ja, können die mNn ohne Probleme gelöscht werden. Als Backup haben wir ja immer das Git Repo.

in gitignore aufnehmen
"empty"-feiles ebenfalls in gitignore

daniel-va · 2025-01-06T06:54:11Z

ui/log_i18next.json

@@ -0,0 +1,85 @@
+{


Ist dies ein pures Logfile? Falls ja, sollte das wahrscheinlich noch ins .gitignore aufgenommen werden.

in gitignore aufnehmen

daniel-va · 2025-01-06T06:54:58Z

ui/package-lock.json

@@ -66,6 +66,7 @@
        "file-loader": "6.2.0",


Das Lock File scheint ebenfalls noch mehrere Merge Conflicts aufzuweisen. Am besten setzt du das manuell auf den Zustand vor dem Merge zurück und führst mal ein npm install aus.

package-lock.json löschen und dann npm install ausführen

daniel-va · 2025-01-06T07:07:35Z

ui/locales/app.de.json

+  "Temperature": "Temperatur",
  "app_cancel_btn_label": "Abbrechen",
-  "by": "von",
+  "by": "__ TO BE TANSLATED __",


Eines der Ziele dieses Issues ist es ja, die Keys/das Konzept der Keys mit Assets & Boreholes abzugleichen. Zu Assets kann ich hier noch einbringen, dass wir dort anstatt von mehreren Prefixes die zusammengehörenden Keys in Namespaces organisieren. Zudem nutzen wir keine Abkürzungen für diese Namespaces, & somit war es bis jetzt nicht nötig, diese irgendwo zu dokumentieren (wie hier im README).

Das ganze könnte hier etwa so aussehen:

{ "camera": { "configuration_header": "Kamera-Einstellungen", "lock": "Sperren" "lock_info_angle": "Im Moment kann im 3D Viewer nur das Azimut verändert werden.", ... }, "toolbox": { "create_section_label": "Schnitte erstellen", "cross_sections": { "depth_label": "Schnitttiefe (m ü. M.)", "format_label": "Format des Schnitts" }, ... }, ... }

Ich würde ausserdem vorschlagen, dass wir die Verwendung der Suffixes (*_label, *_hint) mal überdenken/standardisieren. Ich finde diese als sehr hilfreich, da damit viel schneller klar wir, für was eine bestimmte Übersetzung verwendet wird. Leider werden sie bis jetzt nicht sehr konsequent eingsetzt. Evtl. könnte man sogar noch mehr davon definieren.

Ich weiss nicht, ob man diese Dinge noch innerhalb dieses Issues behandeln möchte. Ansonsten könnte man sich diese Dinge vielleicht mal in einem Folgetasks überlegen.

nOester · 2025-01-09T16:32:31Z

Hallo @daniel-va
Danke für das Review. Gerne würde ich die punkte mit dir nächste Woche besprechen. Ich habe dir ein e-mail mit einem Terminvorschlag gesendet.

oeni2023 and others added 21 commits October 30, 2024 15:39

dependecies installed

1c3d41e

i18next-scanner config integrated

d870388

i18next-scanner installed; n installed for node version managament

3a06f8a

i18next-scanner config file modified

1d4d2f6

loclaes modified

ec6dab7

imports of swissforaged deleted

e0c5fd3

compare and combine created

c8fb3dc

readme modified

bfbd444

translation files

f46022f

compare and combine modified

8d0bfd2

compare and combine modified

5143326

compare and combine modified

44d8c50

readme.md moved

781b149

files moved

ecd3abd

gitignore changed

135ca88

files moved

f3b40d2

files moved

f45caac

Readme.md modified

d39762f

compare and combine modified

236b488

files moved

d203570

Merge branch 'test_viewer' into feature/viewer-1344-locale-keys-berei…

7c2b2a8

…nigen

nOester requested review from daniel-va and vej-ananas December 17, 2024 09:39

nOester linked an issue Dec 17, 2024 that may be closed by this pull request

Prefixes in Locals aktualisieren #1344

Open

5 tasks

daniel-va reviewed Jan 6, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feature/viewer 1344 locale keys bereinigen #1405

Feature/viewer 1344 locale keys bereinigen #1405

nOester commented Dec 17, 2024

sonarqubecloud bot commented Dec 17, 2024

daniel-va Jan 6, 2025

nOester Jan 14, 2025

daniel-va Jan 6, 2025

nOester Jan 14, 2025

daniel-va Jan 6, 2025

nOester Jan 14, 2025

daniel-va Jan 6, 2025

nOester Jan 14, 2025

daniel-va Jan 6, 2025

nOester Jan 14, 2025

daniel-va Jan 6, 2025

nOester Jan 14, 2025

daniel-va Jan 6, 2025

nOester Jan 14, 2025

daniel-va Jan 6, 2025

nOester Jan 14, 2025

daniel-va Jan 6, 2025

nOester Jan 14, 2025

daniel-va Jan 6, 2025

nOester commented Jan 9, 2025

	## 2) Run script to campare and combine translations
	## 2) Run script to compare and combine translations

	- (optional) Modifiy configuration:
	- (optional) Modify configuration:

	"tbx_data-download": "Datendownload",
	"tbx_data_download": "Datendownload",

Feature/viewer 1344 locale keys bereinigen #1405

Are you sure you want to change the base?

Feature/viewer 1344 locale keys bereinigen #1405

Conversation

nOester commented Dec 17, 2024

sonarqubecloud bot commented Dec 17, 2024

Quality Gate passed

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

nOester commented Jan 9, 2025