[PATCH v3 03/10] REST: Add Projects to the API
Andy Doan
andy.doan at linaro.org
Sat May 21 06:12:58 AEST 2016
On 05/19/2016 02:03 AM, Finucane, Stephen wrote:
>> + page_size = settings.REST_RESULTS_PER_PAGE
> nit: This is already a DRF-provided setting that we can override:
>
> http://www.django-rest-framework.org/api-guide/settings/#page_size
>
> However, Given the deprecation of other similar parameters, I'm fine to
> stick with our own parameter (those settings might be worth reading
> though, in case there's anything cool there :)).
I thought PAGE_SIZE was really generic and would wind up causing a
comment line in code. So I figured using our own setting would make it
more clear/obvious.
More information about the Patchwork
mailing list