[RFC 2/2] REST: Add Projects to the REST API
Andy Doan
andy.doan at linaro.org
Fri Mar 25 05:57:28 AEDT 2016
On 03/24/2016 10:47 AM, Finucane, Stephen wrote:
> On 16 Mar 11:13, Andy Doan wrote:
>> This exports projects via the REST API.
>>
>> Security Constraints:
>> * Anyone (logged in or not) can read all projects.
>> * Only admins can create/delete projects.
>> * Project maintainers are allowed to update (ie "patch" project
>> attributes)
>
> Assume you want this reviewed, so comments below. Generally like the
> idea though _if_ we can approach it bit by bit :)
That's my exact plan.
> So I know DRF provides its own 'APITestCase' class. Would it beneficial
> to use this?
Good tip, it will remove some boiler plate.
> * http://www.django-rest-framework.org/api-guide/testing/#test-cases
> * http://stackoverflow.com/q/28221754/613428
>> +class ProjectSerializer(serializers.HyperlinkedModelSerializer):
>> + class Meta:
>> + model = Project
>
> Should we serialize 'tags' also (read only)?
Seems sane.
NOTE: Damien's code only exposes: ('id', 'name', 'linkname',
'listemail', 'web_url', 'scm_url', 'webscm_url'). ie
"send_notifications" and "use_tags" are hidden. I didn't see why that
would be helpful, and it also adds to development costs to have to
remember a field added to models.Project means a change needed for this.
Does this make sense?
More information about the Patchwork
mailing list