gitpab icon indicating copy to clipboard operation
gitpab copied to clipboard

Remove time spent not considered in "Spent time" report

Open wdrechsel opened this issue 6 years ago • 8 comments

While using /remove_time_spent in a Gitlab ticket it is not considered in the "Spent time" report, all former entries should be set to 0 when calculating a sum.

wdrechsel avatar Jan 11 '19 18:01 wdrechsel

Currently I use negative values for it: /spent -1h

mnvx avatar Jan 11 '19 19:01 mnvx

Sure, but there is the option to use /remove_time_spent and if a user does it, the former entries shouldn't be considered.

wdrechsel avatar Jan 11 '19 19:01 wdrechsel

For this purpose must be edited method EloquentNoteService::parseSpentTimeItem().

It must be separated to 2 methods for parsing /spend x and for parsing /remove_time_spent. In case of /remove_time_spent must be calculated amount of time spent by ~~user~~ all users on current issue (using service EloquentSpentService) and must be added record with negative value of this amount.

Also must be edited test with stub of service EloquentSpentService.

I may do it only after 20th february. If anyone may do it early, I will be grateful.

mnvx avatar Jan 11 '19 20:01 mnvx

Not sure if I get it right, but /remove_time_spent removes all time of all users, not only the time of the one who executed this function.

wdrechsel avatar Jan 11 '19 20:01 wdrechsel

I think you are right. I has updated my comment above.

mnvx avatar Jan 11 '19 20:01 mnvx

@mnvx Bug is actual. Any plan for Fixed is ?

KANekT avatar Jul 07 '20 12:07 KANekT

I can find time for it only after summer. Maybe someone else want to do it?

mnvx avatar Jul 07 '20 13:07 mnvx

I can find time for it only after summer. Maybe someone else want to do it?

@mnvx any plan to fix it ?

KANekT avatar Sep 09 '20 08:09 KANekT