Hi guys,
I made some changes to the stormtimetracking module to show (see the attachment) in the timetracking list, not only the "Billing duration" but also the real worked time.

I can create a path and share it, but before I wont to know if someone is interested.

Nicolo'

CommentFileSizeAuthor
#7 storm-[747316].patch8.72 KBnicoloconte
#5 storm-[747316].patch8.73 KBnicoloconte
#2 stormtimetracking.admin_.inc_.patch2.13 KBnicoloconte
#2 stormtimetracking.theme_.inc_.patch6.61 KBnicoloconte
tt.jpg103.64 KBnicoloconte

Comments

juliangb’s picture

juliangb commented

It'd be great if you could share - whether or not we put it into the main core of storm, i'm sure it will provide a great example to people for how they can modify storm.

nicoloconte’s picture

nicoloconte commented
StatusFileSize
new stormtimetracking.theme_.inc_.patch6.61 KB
new stormtimetracking.admin_.inc_.patch2.13 KB

Ok, attached you can find the two patch for the stormtimetracking module!

Nicolo'

nicoloconte’s picture

nicoloconte commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, stormtimetracking.theme_.inc_.patch, failed testing.

nicoloconte’s picture

nicoloconte commented
Status: Needs work » Needs review
StatusFileSize
new storm-[747316].patch8.73 KB

Let's try with this!

juliangb’s picture

juliangb commented

Haven't had a chance to apply yet - but do just watch out, there is trailing whitespace on about 5 lines. Dreditor is great for spotting that sort of thing.

nicoloconte’s picture

nicoloconte commented
StatusFileSize
new storm-[747316].patch8.72 KB

#747316 by nicoloconte | juliangb: I deleted the trailing whitespace. Dreditor is great!!

juliangb’s picture

juliangb commented

Happy from a code style, and code read through for putting into Storm. Haven't applied it yet.

My only wonder is whether users would like this also in the TT list view, or whether it would clutter that more.

Therefore, I now "ask the audience". Please comment with views on whether you'd like this added, or prefer without.

juliangb’s picture

juliangb commented
Status: Needs review » Closed (won't fix)

This patch is a great example for those who want to do this.

Doesn't seem to be too much interest in getting it in though from my request, so will close this issue and we can reconsider if this is something lots of people want.