← Back to team overview

launchpad-reviewers team mailing list archive

[Merge] lp:~stevenk/launchpad/export-archive-status into lp:launchpad

 

Steve Kowalik has proposed merging lp:~stevenk/launchpad/export-archive-status into lp:launchpad.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)
Related bugs:
  Bug #1105541 in Launchpad itself: "Please export the PPA status field in the API"
  https://bugs.launchpad.net/launchpad/+bug/1105541

For more details, see:
https://code.launchpad.net/~stevenk/launchpad/export-archive-status/+merge/146339

Export the status field of IArchive over the API for devel.
-- 
https://code.launchpad.net/~stevenk/launchpad/export-archive-status/+merge/146339
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~stevenk/launchpad/export-archive-status into lp:launchpad.
=== modified file 'lib/lp/soyuz/interfaces/archive.py'
--- lib/lp/soyuz/interfaces/archive.py	2013-01-11 00:18:49 +0000
+++ lib/lp/soyuz/interfaces/archive.py	2013-02-04 05:31:20 +0000
@@ -384,12 +384,11 @@
             description=_("Maximum size, in MiB, allowed for the archive.")))
 
     purpose = Int(
-        title=_("Purpose of archive."), required=True, readonly=True,
-        )
+        title=_("Purpose of archive."), required=True, readonly=True)
 
-    status = Int(
-        title=_("Status of archive."), required=True, readonly=True,
-        )
+    status = exported(
+        Int(title=_("Status of archive."), required=True, readonly=True),
+        as_of='devel')
 
     sources_cached = Int(
         title=_("Number of sources cached"), required=False,

=== modified file 'lib/lp/soyuz/stories/webservice/xx-archive.txt'
--- lib/lp/soyuz/stories/webservice/xx-archive.txt	2012-11-13 13:32:48 +0000
+++ lib/lp/soyuz/stories/webservice/xx-archive.txt	2013-02-04 05:31:20 +0000
@@ -54,6 +54,7 @@
     resource_type_link: u'http://.../#archive'
     self_link: u'http://.../~cprov/+archive/ppa'
     signing_key_fingerprint: None
+    status: u'Active'
     suppress_subscription_notifications: False
     web_link: u'http://launchpad.../~cprov/+archive/ppa'