Merge pull request #17 from kevinhowbrook/16-add-limit-param

Adds limit param and settings so more forms can be fetched

Author: kevinhowbrook

README.md

@@ -24,9 +24,12 @@ You will need an API key from Jotform. Add the following variables to your setti
 WAGTAIL_JOTFORM = {
     "API_KEY": "somekey",
     "API_URL": "https://api.jotform.com",
+    "LIMIT": 50,
 }
 ```
 
+`LIMIT` is the number of results in each result set for form list. Default is 50. Maximum is 1000.
+
 If your Jotform account is in [EU safe mode](https://www.jotform.com/eu-safe-forms/), your `JOTFORM_API_URL` should be `https://eu-api.jotform.com`.
 
 Add the following to your `INSTALLED_APPS` in settings, and note that `wagtail_jotform` depends on `routable_page`:

wagtail_jotform/__init__.py

@@ -1,2 +1,2 @@
-VERSION = (1, 0, 3)
+VERSION = (1, 0, 4)
 __version__ = ".".join(map(str, VERSION))

wagtail_jotform/tests/settings.py

@@ -72,4 +72,5 @@
 WAGTAIL_JOTFORM = {
     "API_KEY": "somekey",
     "API_URL": "https://wagtail-jotform.com",
+    "LIMIT": 50,
 }

wagtail_jotform/tests/tests.py

@@ -160,6 +160,7 @@ def test_settings(self):
         del settings.WAGTAIL_JOTFORM
         self.assertFalse(wagtail_jotform_settings.API_KEY)
         self.assertFalse(wagtail_jotform_settings.API_URL)
+        self.assertFalse(wagtail_jotform_settings.LIMIT)
 
     def test_page_renders_with_bad_settings(self):
         response = self.client.get("/embeded-form-page/")

wagtail_jotform/utils.py

@@ -38,9 +38,9 @@ def fetch_data(url, headers=None, **params):
 
 
 def fetch_jotform_data():
-
+    limit = wagtail_jotform_settings.get("LIMIT", 50)
     headers = {"APIKEY": wagtail_jotform_settings.API_KEY}
-    url = f"{wagtail_jotform_settings.API_URL}/user/forms"
+    url = f"{wagtail_jotform_settings.API_URL}/user/forms?limit={limit}"
 
     return fetch_data(url, headers)