search

kriskbx / gitlab-time-tracker

🦊🕘 A command line interface for GitLab's time tracking feature.
GNU General Public License v2.0
458 stars 82 forks source link

Gtt report output to xlsx format and time_format as seconds, cells are still string #70

Closed daften closed 6 years ago

daften commented 6 years ago

When you want to use the report data in excel, I'd expect numeric data to be a number formatted cell in excel, so I can e.g. use autosum. At the moment when outputting time_format as --time_format="[%seconds_overall]", the output cells are strings and summing a column e.g. with a condition on a certain user, gives me 0 as the total. I could easily create custom reports with totals by user based on the records sheet, if the numbers data was actual numbers. Can this be changed?

cgdobre commented 6 years ago

Thanks for this incentive to get me to overcome my laziness of formatting the cells :)

Yes, I'm sure it's possible, I just made the simplest possible implementation and preferred due to the reason above to just correct the cell type from excel directly.

I'll try to get a PR in during the weekend.

kriskbx commented 6 years ago

Thanks a lot @cgdobre ! Looking forward to merge it :)

kriskbx commented 6 years ago

I just released 1.7.28 with the newly merged fix. Thanks a lot @cgdobre

daften commented 6 years ago

Awesome, thanks @cgdobre and @kriskbx :)