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

Corrections for French and Romanian time display #422

Merged
merged 1 commit into from
Aug 21, 2024

Conversation

clingner
Copy link
Contributor

In the French edition, the "nach" has been removed for "20 nach" and "25 nach" because it is not needed in either case. Correction of the "offsetHour" for "case 35". It showed an hour too early if the "hasThirtyfive" is set.

In der französischen Sprachausgabe darf das «ET»(NACH) bei 20 und 25 Minuten nicht angezeigt werden. «IL EST UNE HEURE ET VINGT» ist falsch. Richtig ist «IL EST UNE HEURE VINGT». Ich habe mit meinen sehr bescheidenen Programmierkenntnisen das probiert zu beheben. Eventuell kommt man auch eleganter zum gleichen Ziel ;-)

Dann ist mir im Fall wo «hasThirtyfive» mit true bewertet ist (rumänische Anzeige) aufgefallen, dass bei einer Uhrzeit z.B. 9:35 die Anzeige 10:35 anzeigt. Hier mussten nur die «offsetHour» richtig plaziert werden. Nun gibt die Anzeige auch 9:35 Uhr aus.

Vielen Dank für die tolle Arbeit hier!

In the French edition, the "nach" has been removed for "20 nach" and "25 nach" because it is not needed in either case.
Correction of the "offsetHour" for "case 35". It showed an hour too early if the "hasThirtyfive" is set.
@dbambus
Copy link
Collaborator

dbambus commented Aug 21, 2024

Hey @clingner,

vielen Dank für deinen Pull Request. Ich denke deine Änderungen machen soweit Sinn. Ich muss das aber nochmal für alle Sprachen mit den Offsethours überprüfen, das ist immer etwas schwierig hier pauschal zu beantworten. Das muss letztlich wie ein Uhrwerk funktionieren :-)

Mit freundlichen Grüßen
David

@dbambus
Copy link
Collaborator

dbambus commented Aug 21, 2024

Okay, ich habe das noch schnell überprüft.

Ergänzung:

  • für die russische Sprachvariante gilt das auch für hasThirtyFive()
  • das hasTwenty() ist in der Ro10x11.hpp auch nicht gesetzt.

Die beiden Fehler werde ich auch noch beheben.

@dbambus dbambus merged commit 2dbc158 into ESPWortuhr:main Aug 21, 2024
1 of 2 checks passed
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.

2 participants