Merge lp:~edwin-grubbs/launchpad/bug-664788-configure-bugtracker-link-permission into lp:launchpad

Proposed by Edwin Grubbs
Status: Merged
Approved by: Curtis Hovey
Approved revision: no longer in the source branch.
Merged at revision: 11925
Proposed branch: lp:~edwin-grubbs/launchpad/bug-664788-configure-bugtracker-link-permission
Merge into: lp:launchpad
Diff against target: 29 lines (+20/-0)
1 file modified
lib/lp/registry/templates/pillar-involvement-portlet.pt (+20/-0)
To merge this branch: bzr merge lp:~edwin-grubbs/launchpad/bug-664788-configure-bugtracker-link-permission
Reviewer Review Type Date Requested Status
Curtis Hovey (community) code + ui Approve
Review via email: mp+40754@code.launchpad.net

Commit message

Allow the bug supervisor to see the "Configure bug tracker" link on the project overview page.

Description of the change

Allow the bug supervisor to see the "Configure bug tracker" link on the project overview page.

To post a comment you must log in.
Revision history for this message
Curtis Hovey (sinzui) wrote :

We talked and IRC and we agreed that bug supervisors not see the progress bar configuration expander because the role can only do the one action.

This looks good to land

review: Approve (code + ui)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'lib/lp/registry/templates/pillar-involvement-portlet.pt'
2--- lib/lp/registry/templates/pillar-involvement-portlet.pt 2010-11-03 14:40:53 +0000
3+++ lib/lp/registry/templates/pillar-involvement-portlet.pt 2010-11-12 21:25:12 +0000
4@@ -31,6 +31,26 @@
5 </tal:disabled>
6 </ul>
7
8+ <tal:pseudo-editor condition="not: context/required:launchpad.Edit">
9+ <tal:comment condition="nothing">
10+ The bug supervisor can view the "Configure bug tracker" link, but
11+ it would be confusing to display the configuration progress
12+ information.
13+ </tal:comment>
14+ <table
15+ tal:condition="context/required:launchpad.BugSupervisor"
16+ style="width: 100%; margin-top: 1em"
17+ id="configuration_links">
18+ <tal:item repeat="link_status view/configuration_links">
19+ <tr>
20+ <td>
21+ <a tal:replace="structure link_status/link/fmt:link" />
22+ </td>
23+ </tr>
24+ </tal:item>
25+ </table>
26+ </tal:pseudo-editor>
27+
28 <tal:editor condition="context/required:launchpad.Edit"
29 define="registration view/registration_completeness">
30