gitea icon indicating copy to clipboard operation
gitea copied to clipboard

Merged PR should hidden time tracker?

Open yp05327 opened this issue 1 year ago • 2 comments

Feature Description

image

As the PR is merged, there's no need to use time tracker.

Screenshots

No response

yp05327 avatar May 16 '24 00:05 yp05327

I would also argue that we shouldn't be able to set a due date on a merged PR since to my knowledge users would only use this feature to ensure that they code review/merge PRs in a timely manner. Not sure if it's used beyond that

kemzeb avatar May 16 '24 02:05 kemzeb

I would also argue that we shouldn't be able to set a due date on a merged PR since to my knowledge users would only use this feature to ensure that they code review/merge PRs in a timely manner. Not sure if it's used beyond that

I think you are talking about a different topic, but I agree, due date on closed PR/issue makes no sense and we should hide that section.

On topic of time tracker: Not sure, maybe some users prefer to retroactively book time into a PR, thought I guess we should not encourage such practives. In fact I think we should automatically stop any active timers when a PR is merged. Maybe keep this behaviour configurable, thought.

silverwind avatar May 27 '24 21:05 silverwind