Skip to content

Conversation

@emre
Copy link

@emre emre commented Jul 24, 2012

No description provided.

@kevinastone
Copy link

+1

@emre
Copy link
Author

emre commented Feb 6, 2013

@jmoiron, could you look at this?

random queries should not be cached.

@kevinastone
Copy link

Yeah, just to expand on my +1, I got bit by this and was forced to switch to @emre 's fork (thanks btw) to continue using Johnny Cache.

What's the hold up in merging this change?

@jself
Copy link
Collaborator

jself commented Feb 6, 2013

Thanks for the submission, most likely we'll be going through all of the bugs again with django 1.5 (and since it involves a bunch of queryset changes, we may have to change a lot). It looks fine, we'll just need to write a test for it and run the full test suite for the versions we'll be supporting in 1.5.

@borgstrom
Copy link

Any update on this one? 1.5 has been out for quite awhile now.

If you need help writing tests for things can you please elaborate on what you would like tested and I'd be happy to help write some tests.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants