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

Already on GitHub? Sign in to your account

Active Record Counter Cache #22

wants to merge 3 commits into


None yet
6 participants

dewyze commented Jan 15, 2013

This is the first pull request I have ever made, so I am sure it needs some work. Please let me know and I would be happy to make any changes.

Additionally, regarding testing, 1. I use Rspec typically, and 2. I don't know how to test for needing to add a column to the taggable schema.

sunny commented Aug 2, 2013

This would be a great patch to add to socialization!


cmer commented Aug 7, 2013

It needs some tests before I can merge... anybody wants to contribute? It'd be a great addition indeed!

vjpr commented Sep 25, 2013

+1 This would be great.

gozali commented Oct 17, 2013



cmer commented Jan 8, 2014

I really want to merge this, but there's no tests yet...

dewyze commented Jan 11, 2014

Admittedly, I know rspec testing, I am not familiar with the rails tests and didn't want to learn them just for this. Can I write rspec tests?

It would be nice to have this patch and some article showing how to update to have counter cache. Thanks.

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