launchpad-reviewers team mailing list archive

[Edwin Grubbs <edwin.grubbs@xxxxxxxxxxxxx>]

Edwin Grubbs has proposed merging lp:~edwin-grubbs/launchpad/bug-664788-configure-bugtracker-link-permission into lp:launchpad/devel.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)
Related bugs:
  #664788 "Configure bug tracker" link doesn't appear on project page for bug supervisor
  https://bugs.launchpad.net/bugs/664788


Allow the bug supervisor to see the "Configure bug tracker" link on the project overview page.
-- 
https://code.launchpad.net/~edwin-grubbs/launchpad/bug-664788-configure-bugtracker-link-permission/+merge/40754
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~edwin-grubbs/launchpad/bug-664788-configure-bugtracker-link-permission into lp:launchpad/devel.

=== modified file 'lib/lp/registry/templates/pillar-involvement-portlet.pt'
--- lib/lp/registry/templates/pillar-involvement-portlet.pt	2010-11-03 14:40:53 +0000
+++ lib/lp/registry/templates/pillar-involvement-portlet.pt	2010-11-12 20:06:14 +0000
@@ -31,6 +31,26 @@
     </tal:disabled>
   </ul>
 
+  <tal:pseudo-editor condition="not: context/required:launchpad.Edit">
+    <!--
+    The bug supervisor can view the "Configure bug tracker" link, but it
+    would be confusing to display the configuration progress
+    information.
+    -->
+    <table
+      tal:condition="view/configuration_links"
+      style="width: 100%; margin-top: 1em"
+      id="configuration_links">
+      <tal:item repeat="link_status view/configuration_links">
+        <tr>
+          <td>
+            <a tal:replace="structure link_status/link/fmt:link" />
+          </td>
+        </tr>
+      </tal:item>
+    </table>
+  </tal:pseudo-editor>
+
   <tal:editor condition="context/required:launchpad.Edit"
               define="registration view/registration_completeness">