Launchpad itself

Merge lp:~brian-murray/launchpad/api-export-messages-count into lp:launchpad

  1. api-export-messages-count
  2. Merge into devel
Proposed by Brian Murray
Status: Merged
Approved by: Brad Crittenden
Approved revision: no longer in the source branch.
Merged at revision: not available
Proposed branch: lp:~brian-murray/launchpad/api-export-messages-count
Merge into: lp:launchpad
Diff against target: 28 lines (+4/-3)
2 files modified
lib/lp/bugs/interfaces/bug.py (+3/-3)
lib/lp/bugs/stories/webservice/xx-bug.txt (+1/-0)
To merge this branch: bzr merge lp:~brian-murray/launchpad/api-export-messages-count
Reviewer Review Type Date Requested Status
Brad Crittenden (community) code Approve
Review via email: mp+22833@code.launchpad.net

Commit message

Export the message_count of a bug in the API.

Description of the change

This branch exports the message_count for a bug in the API.

To post a comment you must log in.
Revision history for this message
Brad Crittenden (bac) wrote :

Brian this change looks fine. I'm curious, though, what happened to the previous branch that made this fix but was never landed?

Also you don't say whether you had a pre-implementation call or not. Just a reminder that any change requires a pre-imp call with a member of the Launchpad team for the specific application.

review: Needs Information (code)
Revision history for this message
Brian Murray (brian-murray) wrote :

Tom and I had tried to land the previous branch with some other API changes to bug that I had made but they ended up conflicting with each other. Tom and I did discuss this at the Launchpad Bugs sprint in Auburn in January.

Revision history for this message
Brad Crittenden (bac) :
review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lib/lp/bugs/interfaces/bug.py'
--- lib/lp/bugs/interfaces/bug.py 2010-03-12 06:24:59 +0000
+++ lib/lp/bugs/interfaces/bug.py 2010-04-05 21:02:31 +0000
@@ -288,9 +288,9 @@
288 number_of_duplicates = exported(288 number_of_duplicates = exported(
289 Int(title=_('The number of bugs marked as duplicates of this bug'),289 Int(title=_('The number of bugs marked as duplicates of this bug'),
290 required=True, readonly=True))290 required=True, readonly=True))
291 message_count = Int(291 message_count = exported(
292 title=_('The number of comments on this bug'),292 Int(title=_('The number of comments on this bug'),
293 required=True, readonly=True)293 required=True, readonly=True))
294 users_affected_count = exported(294 users_affected_count = exported(
295 Int(title=_('The number of users affected by this bug '295 Int(title=_('The number of users affected by this bug '
296 '(not including duplicates)'),296 '(not including duplicates)'),
297297
=== modified file 'lib/lp/bugs/stories/webservice/xx-bug.txt'
--- lib/lp/bugs/stories/webservice/xx-bug.txt 2010-03-24 21:59:44 +0000
+++ lib/lp/bugs/stories/webservice/xx-bug.txt 2010-04-05 21:02:31 +0000
@@ -34,6 +34,7 @@
34 id: 1134 id: 11
35 latest_patch_uploaded: None35 latest_patch_uploaded: None
36 linked_branches_collection_link: u'http://.../bugs/11/linked_branches'36 linked_branches_collection_link: u'http://.../bugs/11/linked_branches'
37 message_count: 7
37 messages_collection_link: u'http://.../bugs/11/messages'38 messages_collection_link: u'http://.../bugs/11/messages'
38 name: None39 name: None
39 number_of_duplicates: 040 number_of_duplicates: 0