rattail/rattail-nationbuilder
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

Limit page size to 100, for fetching Person records from NB API

Browse source 
apparently that is the effective limit..so must honor it
This commit is contained in:
Lance Edgar 2023-10-16 11:12:51 -05:00
parent a7071d140b
commit 6cd71ea4c2
2 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
rattail_nationbuilder/importing/nationbuilder.py
View file

@ -103,7 +103,8 @@ class NationBuilderCachePersonImporter(FromNationBuilder, nationbuilder_importin
] + primary_address_fields ] + primary_address_fields
def get_host_objects(self): def get_host_objects(self):
return self.nationbuilder.get_people(page_size=500) return self.nationbuilder.get_people(page_size=100,
progress=self.progress)
def normalize_host_object(self, person): def normalize_host_object(self, person):

4
rattail_nationbuilder/nationbuilder/webapi.py
View file

@ -95,6 +95,10 @@ class NationBuilderWebAPI(object):
https://apiexplorer.nationbuilder.com/nationbuilder#People https://apiexplorer.nationbuilder.com/nationbuilder#People
""" """
# nb. found this limit in practice, but it is not documented?!
if page_size > 100:
raise ValueError("page_size cannot be more than 100")
response = self.get('/api/v1/people/count') response = self.get('/api/v1/people/count')
count = response.json()['people_count'] count = response.json()['people_count']
pages = count // page_size pages = count // page_size