Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Übersetzungen der Config-Texte korrigiert #83

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

MC-PMOE
Copy link
Contributor

@MC-PMOE MC-PMOE commented Feb 5, 2025

Damit der in den Einstellungen hinterlegte Sprog-key geprüft wird, muss der Wert des Feldes und nicht der Config-Key übergeben werden. Falls die Funktionalität jedoch so gedacht war, dass der tatsächliche Config-Key in Sprog hinterlegt werden muss, so sollte dieser als Notiz am Feld ergänzt werden. Die Hilfe liest sich aber eher so, dass man beliebige Sprog-Keys in den Textefeldern hinterlegen können soll

Damit der in den Einstellungen hinterlegte Sprog-key geprüft wird, muss der Wert des Feldes und nicht der Config-Key übergeben werden.
Falls die Funktionalität jedoch so gedacht war, dass der tatsächliche Config-Key in Sprog hinterlegt werden muss, so sollte dieser als Notiz am Feld ergänzt werden.
Die Hilfe liest sich aber eher so, dass man beliebige Sprog-Keys in den Textefeldern hinterlegen können soll
@alxndr-w
Copy link
Member

alxndr-w commented Feb 5, 2025

Ich erinnere mich ganz dumpf daran, dass etwas nicht so war (oder immer noch ist) wie gedacht.

An sich sollte ganz regulär {{ meinsprogkey }} eingegeben werden, so macht es auch mein Demo-SQL.

Danke schonmal für den PR, ich prüfe das zeitnah.

@alxndr-w alxndr-w self-assigned this Feb 5, 2025
@alxndr-w alxndr-w added the bug Something isn't working label Feb 5, 2025
@alxndr-w alxndr-w modified the milestones: 5.1.0, 5.2.0 Feb 5, 2025
@MC-PMOE
Copy link
Contributor Author

MC-PMOE commented Feb 7, 2025

Bei meinem PR werden die geschweiften Klammern nicht benötigt, ist / war aber glaub ich auch nicht von dir so vorgesehen da du ja direkt die Methoden von Sprog mit "wildcard" nutzt und nicht über den Platzhalter gehst.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants