From 310a89f2feb885417e23a0e188cf0faa94105846 Mon Sep 17 00:00:00 2001 From: Isaac Bythewood Date: Wed, 20 Mar 2013 20:37:04 -0300 Subject: [PATCH 1/2] Change default pin load limit to 50 instead of 30 --- pinry/settings/__init__.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pinry/settings/__init__.py b/pinry/settings/__init__.py index 7dbe482..2553c03 100644 --- a/pinry/settings/__init__.py +++ b/pinry/settings/__init__.py @@ -73,7 +73,7 @@ MESSAGE_TAGS = { messages.SUCCESS: 'alert alert-success', messages.INFO: 'alert alert-info', } -API_LIMIT_PER_PAGE = 30 +API_LIMIT_PER_PAGE = 50 INSTALLED_APPS = ( From 4f8139f6159682246282b47ea2e19f4a8ce0ad51 Mon Sep 17 00:00:00 2001 From: Isaac Bythewood Date: Wed, 20 Mar 2013 21:02:28 -0300 Subject: [PATCH 2/2] Fix bug in sometimes loading the same pins again because the offset is not increased outside of "success" --- pinry/static/js/pinry.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/pinry/static/js/pinry.js b/pinry/static/js/pinry.js index 1facf1f..ca05c7d 100644 --- a/pinry/static/js/pinry.js +++ b/pinry/static/js/pinry.js @@ -136,10 +136,10 @@ $(window).load(function() { $('body').append(theEnd); } } - - // Up our offset, it's currently defined as 30 in our settings - offset += apiLimitPerPage; }); + + // Up our offset, it's currently defined as 50 in our settings + offset += apiLimitPerPage; }