launchpad-reviewers team mailing list archive

[Steve Kowalik <stevenk@xxxxxxxxxx>]

Steve Kowalik has proposed merging lp:~stevenk/launchpad/include-ppa-url into lp:launchpad.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~stevenk/launchpad/include-ppa-url/+merge/64499

Also include the URL to the PPA in the e-mail sent to subscribers when they are subscribed to a private PPA.
-- 
https://code.launchpad.net/~stevenk/launchpad/include-ppa-url/+merge/64499
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~stevenk/launchpad/include-ppa-url into lp:launchpad.

=== modified file 'lib/canonical/launchpad/emailtemplates/ppa-subscription-new.txt'
--- lib/canonical/launchpad/emailtemplates/ppa-subscription-new.txt	2010-12-16 11:07:24 +0000
+++ lib/canonical/launchpad/emailtemplates/ppa-subscription-new.txt	2011-06-14 06:34:31 +0000
@@ -9,6 +9,10 @@
 
 <%(recipient_subscriptions_url)s>
 
+You can find out more about the archive here:
+
+<%(ppa_url)s>
+
 You can find out more about %(registrant_name)s here:
 
 <%(registrant_profile_url)s>

=== modified file 'lib/canonical/launchpad/mailnotification.py'
--- lib/canonical/launchpad/mailnotification.py	2011-06-07 20:16:27 +0000
+++ lib/canonical/launchpad/mailnotification.py	2011-06-14 06:34:31 +0000
@@ -303,6 +303,7 @@
             'ppa_displayname': ppa_displayname,
             'ppa_reference': ppa_reference,
             'ppa_description_blurb': description_blurb,
+            'ppa_url': canonical_url(archive),
             'recipient_subscriptions_url': recipient_subscriptions_url,
             }
         body = MailWrapper(72).format(template % replacements,

=== modified file 'lib/lp/soyuz/doc/archivesubscriber.txt'
--- lib/lp/soyuz/doc/archivesubscriber.txt	2010-12-17 06:04:39 +0000
+++ lib/lp/soyuz/doc/archivesubscriber.txt	2011-06-14 06:34:31 +0000
@@ -133,6 +133,10 @@
     <BLANKLINE>
     <http://launchpad.dev/~joesmith/+archivesubscriptions>
     <BLANKLINE>
+    You can find out more about the archive here:
+    <BLANKLINE>
+    <http://launchpad.dev/~cprov/+archive/p3a>
+    <BLANKLINE>
     You can find out more about Celso Providelo here:
     <BLANKLINE>
     <http://launchpad.dev/~cprov>
@@ -169,6 +173,10 @@
     <BLANKLINE>
     <http://launchpad.dev/~fredsmith/+archivesubscriptions>
     <BLANKLINE>
+    You can find out more about the archive here:
+    <BLANKLINE>
+    <http://launchpad.dev/~cprov/+archive/p3a>
+    <BLANKLINE>
     You can find out more about Celso Providelo here:
     <BLANKLINE>
     <http://launchpad.dev/~cprov>