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.

Sign up for GitHub

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

Jump to bottom

Time tracking: used time reported in days but length of the day not configurable #33333

Closed
mkoskl opened this issue Jan 20, 2025 · 3 comments
Closed

Time tracking: used time reported in days but length of the day not configurable #33333

mkoskl opened this issue Jan 20, 2025 · 3 comments
Labels
type/proposal The new feature has not been accepted yet but needs to be discussed first.
Milestone
1.23.2

Comments

@mkoskl
Copy link

mkoskl commented Jan 20, 2025

Feature Description

Hello,

this issue is related to #23112

The used time is reported (if time goes over 24 hours) in days, e.g. "1 day 4 hours".

But it's a bit confusing since work day length is not same as 24h.

The time estimate part reports better, like "48 hours".

Could the used time part report as hours and minutes like the estimate part does?

Br, Miika

Screenshots

No response
@mkoskl mkoskl added the type/proposal The new feature has not been accepted yet but needs to be discussed first. label Jan 20, 2025
@wxiaoguang
Copy link
Contributor

wxiaoguang commented Jan 20, 2025 •
edited
Loading

It should have been fixed by Make tracked time representation display as hours #33315 in 1.24

Not sure whether we should backport it to 1.23 (consider it as a bug fix?)

@wxiaoguang wxiaoguang mentioned this issue Jan 20, 2025
Merged
@wxiaoguang
Copy link
Contributor

wxiaoguang commented Jan 20, 2025 •
edited
Loading

Try to backport the fix to 1.23: Make tracked time representation display as hours (#33315) #33334

@lunny lunny added this to the 1.23.2 milestone Jan 20, 2025
lunny pushed a commit that referenced this issue Jan 20, 2025
@wxiaoguang
Make tracked time representation display as hours (#33315) (#33334)
eef6355 
Try to backport #33315, the only trivial conflict is in the helper
functions map in the helper.go

Fix #33333

Co-authored-by: Sysoev, Vladimir <[email protected]>
@wxiaoguang
Copy link
Contributor

Try to backport the fix to 1.23: Make tracked time representation display as hours (#33315) #33334

The fix is in 1.23-nightly now (it is a stable release and will be 1.23.2 soon)

Feel free to try

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type/proposal The new feature has not been accepted yet but needs to be discussed first.
Projects
None yet
Milestone
1.23.2
Development

No branches or pull requests
3 participants
@lunny @wxiaoguang @mkoskl