Merge lp:~cjohnston/summit/scheduler-tools into lp:summit

Proposed by Chris Johnston
Status: Merged
Approved by: Chris Johnston
Approved revision: 414
Merge reported by: Chris Johnston
Merged at revision: not available
Proposed branch: lp:~cjohnston/summit/scheduler-tools
Merge into: lp:summit
Diff against target: 58 lines (+17/-4)
3 files modified
summit/schedule/templates/schedule/admin.html (+8/-0)
summit/schedule/templates/schedule/meeting.html (+8/-4)
summit/schedule/tests.py (+1/-0)
To merge this branch: bzr merge lp:~cjohnston/summit/scheduler-tools
Reviewer Review Type Date Requested Status
Michael Hall (community) Approve
Review via email: mp+109539@code.launchpad.net

Commit message

Adds admin info to the meeting page

To post a comment you must log in.
Revision history for this message
Michael Hall (mhall119) wrote :

Looks okay, but why is the theme_media line being added in this MP?

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== added file 'summit/schedule/templates/schedule/admin.html'
2--- summit/schedule/templates/schedule/admin.html 1970-01-01 00:00:00 +0000
3+++ summit/schedule/templates/schedule/admin.html 2012-06-10 20:38:20 +0000
4@@ -0,0 +1,8 @@
5+<div class="action-links">
6+ <p style="font-size: 16px; font-weight: bold;">Admin Info</p>
7+ <ul>
8+ <li><strong>Created by:</strong> {{ meeting.drafter.name }}</li>
9+ <li><strong>Approved by:</strong> {{ meeting.approver.name }}</li>
10+ <li><strong>Scheduled:</strong> {% for ai in agenda_items %}{% if ai.auto %}Auto{% else %}Manual{% endif %}{% endfor %}</li>
11+ </ul>
12+</div>
13
14=== modified file 'summit/schedule/templates/schedule/meeting.html'
15--- summit/schedule/templates/schedule/meeting.html 2012-05-06 00:02:48 +0000
16+++ summit/schedule/templates/schedule/meeting.html 2012-06-10 20:38:20 +0000
17@@ -1,5 +1,5 @@
18 {% extends "base.html" %}
19-{% load datetime markup %}
20+{% load schedule_perms datetime markup %}
21
22 {% block page_name %}
23 {{ meeting.title }} -
24@@ -107,15 +107,19 @@
25 </div>
26 <div class="row">
27 {% ifnotequal meeting.type 'plenary' %}
28- <div id="Attendees" class="span-7">
29+ <div id="Attendees" class="span-5">
30 <h3>Attendees</h3>
31 {% for attendee in attendees %}
32 <a href="http://launchpad.net/~{{ attendee.user.username }}">{% if attendee.required %}<strong>{% endif %}{{ attendee.name }}{% if attendee.required %}</strong>{% endif %}</a>{% if not forloop.last %}, {% endif %}
33 {% endfor %}
34 </div>
35 {% endifnotequal %}
36-
37- <div id="Links" class="span-5 last">
38+ <div class="span-4">
39+{% if summit_organizer %}
40+ {% include "schedule/admin.html" %}
41+{% endif %}
42+ </div>
43+ <div id="Links" class="span-3 last">
44 <h3>Links</h3>
45 <ul>
46 <li><a href="{{meeting.link_to_pad}}" target="_new">Notes in a separate window</a></li>
47
48=== modified file 'summit/schedule/tests.py'
49--- summit/schedule/tests.py 2012-02-25 01:28:31 +0000
50+++ summit/schedule/tests.py 2012-06-10 20:38:20 +0000
51@@ -42,6 +42,7 @@
52 factory.default_mapping[NameField] = str
53
54 site_root = getattr(settings, 'SITE_ROOT', 'http://summit.ubuntu.com')
55+theme_media = getattr(settings, 'THEME_MEDIA', 'ubuntu_website.MEDIA_ROOT')
56
57 class RequestFactory(djangotest.Client):
58 """

Subscribers

People subscribed via source and target branches