gitpab icon indicating copy to clipboard operation
gitpab copied to clipboard

Spent time - Spent time should be counted by the spent_at column, not by created_at

Open danielciobanica opened this issue 5 years ago • 11 comments

We can add time spent on gitlab Today(2020-08-28), for last week (e.g. /spend 4h 2020-08-20).

Reports will count the hours by the created_at date which is Today (2020-08-28)

I think we should change this because the and also update the filter to search for spent_at time.

danielciobanica avatar Aug 28 '20 13:08 danielciobanica

@danielciobanica Thanks for involve, please read similar issue #36

mnvx avatar Aug 28 '20 13:08 mnvx

ok, i guess we can close this issue. Btw, i thought it's an open source repository. Is there any chance to push my code?

danielciobanica avatar Aug 28 '20 14:08 danielciobanica

You are may fork of the project and create pull request

mnvx avatar Aug 28 '20 14:08 mnvx

ok, i'll do that

danielciobanica avatar Aug 28 '20 14:08 danielciobanica

I am following this, as I have same feature request :).. thank you -> still using "created_at" time and not "spend_at" time

KeldNielsen86 avatar Nov 19 '22 23:11 KeldNielsen86

https://github.com/danielciobanica/gitpab

@KeldNielsen86 - You can clone the project from here. I forked it and pushed my changes to use the spend_at time

danielciobanica avatar Nov 28 '22 14:11 danielciobanica

https://github.com/danielciobanica/gitpab

@KeldNielsen86 - You can clone the project from here. I forked it and pushed my changes to use the spend_at time

Hi @danielciobanica

thank you, what else is the difference between your projects?. (I prefer just to update this project) as i had many problems installing this project on my server.)

isen 't it possible to help @mnvx in order to change this in this project? :)

thank you

regards Keld Nielsen

KeldNielsen86 avatar Nov 29 '22 11:11 KeldNielsen86

@KeldNielsen86 - nothing else is different.

I created a pull-request in the past but it wasn't approved. You can read why here:

https://github.com/zubroide/gitpab/pull/64

@mnvx calculates employees salaries based on the time spent and he wants to make sure employee can't add time spent in the past

danielciobanica avatar Nov 29 '22 11:11 danielciobanica

Ahh. I see. it makes sense. (good point). If my developers register time which in the previous payment period, they will lose those hours. ( else I like the idea that they are able to register hours spend for instance yesterday, in case they forgot.

thank you for the answer

KeldNielsen86 avatar Nov 29 '22 11:11 KeldNielsen86

Yes, it depends on what are you using this time report for. But everybody may forget at some point to register the time spent, so that's why i needed the report based on spent_at instead of created_at

danielciobanica avatar Nov 29 '22 11:11 danielciobanica