search

joashivanmoodley / django-pagination

Automatically exported from code.google.com/p/django-pagination
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

autopaginate second parameter should be a variable #4

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
would be nice if we could do:

autopaginate  object_list number_of_items 

where number_of_items can be a variable defined in the context or a number.

Should I create a patch and submit?

Original issue reported on code.google.com by lfrodrig...@gmail.com on 16 Jul 2008 at 1:13

GoogleCodeExporter commented 8 years ago 
Great idea!  Definitely submit a patch. I'd love to have that functionality.

Original comment by flo...@gmail.com on 21 Jul 2008 at 8:56

GoogleCodeExporter commented 8 years ago 
Hello,

Here's the path. 
You should try to include this on contrib of django since it's really usefull.

Original comment by lfrodrig...@gmail.com on 21 Jul 2008 at 1:40

Attachments:

GoogleCodeExporter commented 8 years ago

Original comment by leidel on 19 Sep 2008 at 9:48

GoogleCodeExporter commented 8 years ago 
I could use that at the moment, ok if I check in?

Original comment by leidel on 19 Sep 2008 at 9:50

GoogleCodeExporter commented 8 years ago 
This looks good, I simply didn't see this ticket :)  The patch will need a bit 
of updating to latest trunk django-
pagination.  I'll apply it after Pinax 0.5 release.

Original comment by flo...@gmail.com on 20 Oct 2008 at 12:48

GoogleCodeExporter commented 8 years ago 
Added in r41.  Thanks, lfrodrigues, you rock!

Original comment by flo...@gmail.com on 24 Oct 2008 at 8:30