caktus/django-timepiece — comment on commit a3652a58400a11c1db698d8e6cc700aba6f69311
File: timepiece/utils.py line 420
By copelco:

I feel like we're bleeding the lines between templates and tags here. I think it's unconventional to yield a label here which we then check for in the template. What do you think of making totals represented in the data structure itself, rather than an extra row that's yielded? For example, you'd use it in the template like (psudeo-code):

for entries, totals in project_totals:
for entry in entries:
print entry
print totals

Just an initial observation though. What do you think?