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

Planner: replace weekday with day #17564

Merged
merged 5 commits into from
Dec 3, 2024

Conversation

Maschga
Copy link
Contributor

@Maschga Maschga commented Dec 3, 2024

Dieser PR ersetzt Weekday(s) durch Day(s), wie in #16091 (comment) geschrieben.

@naltatis
Copy link
Member

naltatis commented Dec 3, 2024

@Maschga oh, da haben wir uns glaube ich missverstanden. Im Code würde ich auf jeden Fall bei weekday bleiben. Das ist inhaltlich auch nicht falsch. Die Go-API nutzt das ja bspw auch genau so wie wir: https://pkg.go.dev/time#Time.Weekday

In dem Kommentar ging es ja "nur" um den allgemeinen englischen Sprachgebrauch im Alltag. Also rein das Label in der UI. Hier könnten wir auf Days gehen. Wobei das für mich auch kein so klarer Fall ist, wie es der Kommentarautor schreibt. Im Deutschen ist Wochentag glaube ich nicht missverständlich. Siehe Wikipedia.

@Maschga
Copy link
Contributor Author

Maschga commented Dec 3, 2024

Das macht es definitiv einfacher. 😂

Wobei das für mich auch kein so klarer Fall ist, wie es der Kommentarautor schreibt. Im Deutschen ist Wochentag glaube ich nicht missverständlich.
Ja, das würde ich so auch sehen.

Dann mache ich hier zu und ändere es in Weblate?

@naltatis
Copy link
Member

naltatis commented Dec 3, 2024

Das Ändern des englischen Labels muss als PR passieren.

@naltatis
Copy link
Member

naltatis commented Dec 3, 2024

Ich würd nur den englischen Value ändern. Auf der Code-Seite sollten wir konsistent bei weekdays bleiben.

@naltatis naltatis changed the title replace weekday with day Planner: replace weekday with day Dec 3, 2024
@Maschga
Copy link
Contributor Author

Maschga commented Dec 3, 2024

Done. 👍

@Maschga Maschga marked this pull request as ready for review December 3, 2024 19:59
@naltatis naltatis merged commit 48da891 into evcc-io:master Dec 3, 2024
6 checks passed
@RTTTC
Copy link
Contributor

RTTTC commented Dec 3, 2024

In English a word weekday in essence equals "workday"

Easy to check, just google weekday dictionary and check any of the top university dictionary definitions. https://dictionary.cambridge.org/dictionary/english/weekday
Holds true for US and UK english at least..

@Maschga Thank you for changing the source EN translation.

@Maschga Maschga deleted the replace-weekday-with-day branch December 5, 2024 18:46
jonilala796 pushed a commit to jonilala796/evcc that referenced this pull request Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants