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
=== 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 21:25:12 +0000
@@ -31,6 +31,26 @@
31 </tal:disabled>31 </tal:disabled>
32 </ul>32 </ul>
3333
34 <tal:pseudo-editor condition="not: context/required:launchpad.Edit">
35 <tal:comment condition="nothing">
36 The bug supervisor can view the "Configure bug tracker" link, but
37 it would be confusing to display the configuration progress
38 information.
39 </tal:comment>
40 <table
41 tal:condition="context/required:launchpad.BugSupervisor"
42 style="width: 100%; margin-top: 1em"
43 id="configuration_links">
44 <tal:item repeat="link_status view/configuration_links">
45 <tr>
46 <td>
47 <a tal:replace="structure link_status/link/fmt:link" />
48 </td>
49 </tr>
50 </tal:item>
51 </table>
52 </tal:pseudo-editor>
53
34 <tal:editor condition="context/required:launchpad.Edit"54 <tal:editor condition="context/required:launchpad.Edit"
35 define="registration view/registration_completeness">55 define="registration view/registration_completeness">
3656