First, thank you for this module.
Second, a feature request. We're using this module to for project/issue/case nodes, but with more then 1 user.
I like to see how many time each user had working on a specific node.

So, I need the options
- More then one user can use the node time tracker at the same time on the same node
- A row in the view list which user(s) had working on the node for that specific time.

Keep up the good work!

CommentFileSizeAuthor
#3 track_per_user_05.patch2.22 KBsanduhrs
#1 track_per_user_01.patch2.23 KBsanduhrs

Comments

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
commented
Assigned: Unassigned » sanduhrs
Status: Active » Needs review
StatusFileSize
new track_per_user_01.patch2.23 KB

More then one user can use the node time tracker at the same time on the same node

Patch attached for this one, please review.

mlncn’s picture

mlncn commented

Thanks for your patches, sanduhrs!

I'm trying to check out all of them, running into this on this patch:

patch < track_per_user_01.patch.txt
patching file node_time_tracker.module
Hunk #2 succeeded at 265 with fuzz 2.
Hunk #3 FAILED at 291.
1 out of 4 hunks FAILED -- saving rejects to file node_time_tracker.module.rej

It should apply against the only release available for download, right?

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
commented
StatusFileSize
new track_per_user_05.patch2.22 KB

Hi Benjamin,
I'm not shure why that happens, but removing the two blank lines I introduced apparently solves the problem - at least for me.
Please try the new patch.
I'm patching against HEAD, as it seems the most advanced version.
vg

robertgarrigos’s picture

robertgarrigos commented

I'm also confused about all these patches for this module. I don't know how to apply them. Apparently some of these need to be applied over this module already patched with some other of your patches, what makes necessary to know the order to apply them.

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
commented

Unless stated otherwise, the patches are all standalone against HEAD.
I'm using Eclipse to generate and apply them and they work flawlessly for me.

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
commented
Status: Needs review » Fixed

As this functionality is really essential and the patch is small and straightforward - commited to HEAD.
Thanks.

mlncn’s picture

mlncn commented

Awesome, sanduhrs! (Sorry I didn't get back to review.)

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.