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

WIP: Replaced Sec2Time with Sec2Hours for better timesheets reporting. #28312

Closed
wants to merge 5 commits into from

Conversation

vsysoev
Copy link
Contributor

@vsysoev vsysoev commented Dec 1, 2023

Current situation.
Tracked time informations is displayed in years, monthes, days, hours, minutes. This is not suitable for timesheets reporting. For example work day is 8 hours, work week often has 5 work days.
What is changed.
Added function Sec2Hours which shows tracked time in hours and minutes which is more convenient while reporting timesheets to management.

Current situation.
Tracked time informations is displayed in years, monthes, days, hours, minutes.
This is not suitable for timesheets reporting. For example work day is 8 hours,
work week often has 5 work days.
What is changed.
Added function Sec2Hours which shows tracked time in hours and minutes which is more
convenient while reporting timesheets to management.

Signed-off-by: Sysoev, Vladimir <i@vsysoev.ru>
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Dec 1, 2023
@pull-request-size pull-request-size bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Dec 1, 2023
@JakobDev
Copy link
Contributor

JakobDev commented Dec 1, 2023

I think this should be configurable on a Repo/Org basis. Not everyone use Gitea the same Way as you do.

@KN4CK3R
Copy link
Member

KN4CK3R commented Dec 1, 2023

There is already a PR for that: #25213

@vsysoev vsysoev changed the title Replaced Sec2Time with Sec2Hours for better timesheets reporting. WIP:Replaced Sec2Time with Sec2Hours for better timesheets reporting. Dec 2, 2023
@vsysoev vsysoev changed the title WIP:Replaced Sec2Time with Sec2Hours for better timesheets reporting. WIP: Replaced Sec2Time with Sec2Hours for better timesheets reporting. Dec 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants