Merge lp:~nigelbabu/summit/779769-remove-attendees-plenary into lp:summit

Proposed by Nigel Babu
Status: Merged
Approved by: Chris Johnston
Approved revision: 117
Merged at revision: 118
Proposed branch: lp:~nigelbabu/summit/779769-remove-attendees-plenary
Merge into: lp:summit
Diff against target: 18 lines (+2/-0)
1 file modified
summit/schedule/templates/schedule/meeting.html (+2/-0)
To merge this branch: bzr merge lp:~nigelbabu/summit/779769-remove-attendees-plenary
Reviewer Review Type Date Requested Status
James Westby (community) Approve
Summit Hackers Pending
Review via email: mp+63264@code.launchpad.net

Description of the change

Remove the Attendees list in the meeting page if the meeting is a plenary.

To post a comment you must log in.
Revision history for this message
James Westby (james-w) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'summit/schedule/templates/schedule/meeting.html'
2--- summit/schedule/templates/schedule/meeting.html 2011-05-09 17:09:28 +0000
3+++ summit/schedule/templates/schedule/meeting.html 2011-06-02 17:19:27 +0000
4@@ -25,12 +25,14 @@
5
6 <iframe width=100% height=700 src="{{ meeting.link_to_pad }}"></iframe>
7
8+ {% ifnotequal meeting.type 'plenary' %}
9 <div id="Attendees" style="float:left;width:60%;">
10 <h3>Attendees</h3>
11 {% for participant in participants %}
12 <a href="http://launchpad.net/~{{ participant.attendee }}">{% if participant.required %}<strong>{% endif %}{{ participant.attendee }}{% if participant.required %}</strong>{% endif %}</a>{% if forloop.last %}{% else %}, {% endif %}
13 {% endfor %}
14 </div>
15+ {% endifnotequal %}
16
17 <div id="Links" style="float:left;width:40%;">
18 <h3>Links</h3>

Subscribers

People subscribed via source and target branches