Merge lp://staging/~cjohnston/ubuntu-ci-services-itself/ts-ticket-more-write-api into lp://staging/ubuntu-ci-services-itself
Proposed by
Chris Johnston
| Status: | Merged |
|---|---|
| Approved by: | Chris Johnston |
| Approved revision: | 58 |
| Merged at revision: | 60 |
| Proposed branch: | lp://staging/~cjohnston/ubuntu-ci-services-itself/ts-ticket-more-write-api |
| Merge into: | lp://staging/ubuntu-ci-services-itself |
| Diff against target: |
321 lines (+176/-21) 6 files modified
docs/components/ticket-system.rst (+30/-9) ticket_system/project/admin.py (+1/-2) ticket_system/ticket/api.py (+7/-3) ticket_system/ticket/tests/test_full_read_api.py (+2/-2) ticket_system/ticket/tests/test_read_api.py (+4/-4) ticket_system/ticket/tests/test_write_api.py (+132/-1) |
| To merge this branch: | bzr merge lp://staging/~cjohnston/ubuntu-ci-services-itself/ts-ticket-more-write-api |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Francis Ginther | Approve | ||
|
Review via email:
|
|||
Commit message
Add a write API to the TS SubTicket/
Description of the change
- Removed set ticket status and set subticket status as they will both be done on a pull basis for phase 0.
- Added write API to SubTicket, Artifact, and SPU resources
- Updated docs
To post a comment you must log in.
Revision history for this message
| Chris Johnston (cjohnston) wrote | # |
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The API looks reasonable for what was discussed in IRC. I would recommend an example if someone else was working on the CLI, but in this case it's not needed.
I noticed a few doc rendering issues caused by a missing line between '::' and the following code snippet.