Fixing Counter and adding a few default counters into middleware #3

Merged
merged 4 commits into from Feb 15, 2013

Conversation

Projects
None yet
2 participants
Contributor

dlamotte commented Sep 21, 2012

The Counter() class was being called with .incr() instead of .increment() which was breaking the counter.

Also, I think it's relatively interesting and easy to simply store how many page hits a specific view is getting. I'm considering adding app specific counters. Also, a site on the whole counter is useful.

Maybe number unique users at some point?

WoLpH merged commit 000edfb into WoLpH:master Feb 15, 2013

Owner

WoLpH commented Feb 15, 2013

My apologies for not merging this sooner, I just noticed the pull request now and it seems that I never got a mail for it...

Owner

WoLpH commented Feb 15, 2013

Thank you for the patch, very nice :)

Contributor

dlamotte commented Feb 15, 2013

Sure, no worries. Thanks for the merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment