Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Don't store count_ref for every hit #83

Closed
arp242 opened this issue Dec 20, 2019 · 0 comments · Fixed by #148
Closed

Don't store count_ref for every hit #83

arp242 opened this issue Dec 20, 2019 · 0 comments · Fixed by #148
Labels
enhancement

Comments

@arp242
Copy link
Owner

arp242 commented Dec 20, 2019

We only record this as a rough indication to check whether or not it's used a commercial site. Can just make a table which increments a counter; this would also allow adding an audit column.
@arp242 arp242 added this to the Version 1.1 milestone Dec 20, 2019
arp242 added a commit that referenced this issue Jan 27, 2020
@arp242
Remove hits.count_ref, add usage table
4268ab3 
TODO: actually increment counter

Fixes #83
@arp242 arp242 mentioned this issue Jan 27, 2020
Merged
arp242 added a commit that referenced this issue Jan 29, 2020
@arp242
Remove hits.count_ref, add usage table
90edb16 
TODO: actually increment counter

Fixes #83
arp242 added a commit that referenced this issue Jan 29, 2020
@arp242
Remove hits.count_ref, add usage table (#148)
252dd30 
* Remove hits.count_ref, add usage table

TODO: actually increment counter

Fixes #83

* No longer need contact list now that we have updates

* Increment counter

I'll add summing up of values later, when it's actually needed (if at
all, we'll see how long we can go with the sum()).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove hits.count_ref, add usage table arp242/goatcounter
1 participant
@arp242