Merge lp://staging/~cjohnston/ubuntu-ci-services-itself/people-write-api into lp://staging/ubuntu-ci-services-itself

Proposed by Chris Johnston
Status: Merged
Approved by: Francis Ginther
Approved revision: 27
Merged at revision: 33
Proposed branch: lp://staging/~cjohnston/ubuntu-ci-services-itself/people-write-api
Merge into: lp://staging/ubuntu-ci-services-itself
Diff against target: 226 lines (+112/-16)
4 files modified
ci-utils/ci_utils/tastypie/test.py (+8/-2)
docs/components/ticket-system.rst (+46/-3)
ticket_system/people/api.py (+5/-3)
ticket_system/people/tests.py (+53/-8)
To merge this branch: bzr merge lp://staging/~cjohnston/ubuntu-ci-services-itself/people-write-api
Reviewer Review Type Date Requested Status
Francis Ginther Approve
Andy Doan (community) Approve
Review via email: mp+198778@code.staging.launchpad.net

Commit message

Add a write API for adding people, update docs with API examples

Description of the change

- Made changes to ci_utils.tastypie.test to use assertHttpCreated/assertHttpAccepted instead of assertEqual and a status_code
- Added a delete helper to ci_utils.tastypie.test
- Added write API to people app
- Updated docs to include API calls

RTD can be viewed at: http://162.213.34.2/rtd/components/ticket-system.html

To post a comment you must log in.
Revision history for this message
Andy Doan (doanac) :
review: Approve
Revision history for this message
Francis Ginther (fginther) wrote :

Approve.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches