Merge lp://staging/~thumper/launchpad/yet-more-api-exports into lp://staging/launchpad

Proposed by Tim Penhey
Status: Merged
Approved by: Paul Hummer
Approved revision: no longer in the source branch.
Merged at revision: not available
Proposed branch: lp://staging/~thumper/launchpad/yet-more-api-exports
Merge into: lp://staging/launchpad
Diff against target: 87 lines (+25/-1)
3 files modified
lib/lp/code/interfaces/codereviewcomment.py (+9/-1)
lib/lp/code/model/codereviewcomment.py (+10/-0)
lib/lp/code/stories/webservice/xx-branchmergeproposal.txt (+6/-0)
To merge this branch: bzr merge lp://staging/~thumper/launchpad/yet-more-api-exports
Reviewer Review Type Date Requested Status
Paul Hummer (community) code Approve
Review via email: mp+23905@code.staging.launchpad.net

Commit message

Expose the date created and author of code review comments over the api.

Description of the change

A very simple branch that exposes two more attributes of the code review comment.

Since we are still trying out how to best use the launchpadlib in tests, I've just added the attributes to the current webservice doctest.

tests:
  stories/webservice/xx-branchmergeproposal.txt

To post a comment you must log in.
Revision history for this message
Paul Hummer (rockstar) :
review: Approve (code)

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.