Hinweis, dass wir Nachrichten weiterleiten können #2

Merged
ldr merged 3 commits from note_external_recipients into master 2025-04-09 20:08:56 +02:00
Owner
No description provided.
dam changed title from Hinweis, dass wir Nachrichten weiterleiten können to WIP: Hinweis, dass wir Nachrichten weiterleiten können 2025-02-27 17:38:13 +01:00
dam requested review from ldr 2025-02-27 17:38:20 +01:00
dam changed title from WIP: Hinweis, dass wir Nachrichten weiterleiten können to Hinweis, dass wir Nachrichten weiterleiten können 2025-03-25 23:34:41 +01:00
dam requested review from yorik 2025-03-25 23:34:50 +01:00
yorik left a comment

Der Text und die Übersetzung sehen gut aus, ich würde aber nicht die benutzen, nur weil ein Link nicht richtig formatiert war

Der Text und die Übersetzung sehen gut aus, ich würde aber nicht `die` benutzen, nur weil ein Link nicht richtig formatiert war
index.php Outdated
@ -5,0 +4,4 @@
$lang = $_GET["lang"] ?? ($lang = "de");
if ($lang !== "de" && $lang !== "en") {
http_response_code(404);
die("Error: Language not found!");
Member

Ist das wirklich gewollt? Ich würde eher per default auf deutsch (oder meinetwegen Englisch) wechseln und maximal einen Hinweis ergänzen, dass die Sprache nicht gefunden wurde und deswegen jetzt auf deutsch gewechselt wurde (oder eben auch den Hinweis einfach ignorieren und dann ist das halt so)

Ist das wirklich gewollt? Ich würde eher per default auf deutsch (oder meinetwegen Englisch) wechseln und maximal einen Hinweis ergänzen, dass die Sprache nicht gefunden wurde und deswegen jetzt auf deutsch gewechselt wurde (oder eben auch den Hinweis einfach ignorieren und dann ist das halt so)
dam marked this conversation as resolved
dam requested review from yorik 2025-04-01 22:35:36 +02:00
ldr approved these changes 2025-04-09 20:07:27 +02:00
ldr left a comment

LGTU
LGTM

co-authored-by @yorik

LGTU LGTM co-authored-by @yorik
ldr merged commit 99f4df672f into master 2025-04-09 20:08:56 +02:00
ldr deleted branch note_external_recipients 2025-04-09 20:08:56 +02:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
FS-InfMath/kummerkasten-php!2
No description provided.