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

Wrong format token for minutes in "Show current time" #239

Closed
fnsign opened this issue Oct 15, 2021 · 2 comments
Closed

Wrong format token for minutes in "Show current time" #239

fnsign opened this issue Oct 15, 2021 · 2 comments
Labels
type: bug Something isn't working

Comments

@fnsign
Copy link

fnsign commented Oct 15, 2021

The default entry in "Show current time" is given as "HH:MM". This leads to a wrong display of the current time as the token MM shows the current month as decimal. Meaning the minutes seem to be stuck at "10" (October). The correct format token is "mm" (lower case).
The default value should be corrected to "HH:mm" instead to display minutes correctly and avoid irritations for the users.

See also https://momentjs.com/docs/#/displaying/format/

@OllisGit OllisGit added status: inNextRelease Will be implemented/fixed in next release type: bug Something isn't working labels Oct 15, 2021
OllisGit added a commit that referenced this issue Oct 15, 2021
- B #239 wrong default time format
@OllisGit
Copy link
Owner

Upps!

@fnsign thanks for the bug-report. It is now fixed in latest version 1.27.1.

@OllisGit OllisGit added status: waitingForTestFeedback and removed status: inNextRelease Will be implemented/fixed in next release labels Oct 15, 2021
@fnsign
Copy link
Author

fnsign commented Oct 15, 2021

Thanks for fixing! The latest release 1.27.1 works now as expected. I'll close the issue.

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

No branches or pull requests

2 participants