Merge lp:~lderan/ubuntu-bots/meeetingology-output into lp:~ubuntu-bots/ubuntu-bots/meetingology

Proposed by Thomas Molloy
Status: Superseded
Proposed branch: lp:~lderan/ubuntu-bots/meeetingology-output
Merge into: lp:~ubuntu-bots/ubuntu-bots/meetingology
Diff against target: 33 lines (+3/-3)
2 files modified
meeting.py (+1/-1)
writers.py (+2/-2)
To merge this branch: bzr merge lp:~lderan/ubuntu-bots/meeetingology-output
Reviewer Review Type Date Requested Status
Ubuntu IRC Bots Pending
Review via email: mp+205424@code.launchpad.net

This proposal has been superseded by a proposal from 2014-02-07.

Description of the change

Changed the layout for the vote output and set Jose to be the one notified when a meeting ends

To post a comment you must log in.
19. By Alan Bell

accepting merge

20. By Alan Bell

accepting fix

21. By Alan Bell

accpeting merge

22. By Thomas Molloy

DONE items for MoinMoin outputs now show up as DONE instead of ACTION

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'meeting.py'
2--- meeting.py 2014-02-07 16:06:41 +0000
3+++ meeting.py 2014-02-07 17:18:24 +0000
4@@ -90,7 +90,7 @@
5 "\n"
6 "Minutes: %(urlBasename)s.moin.txt")
7 endMeetingNotification = ("Meeting in %(channel)s has just ended")
8- endMeetingNotificationList = ["lderan"]
9+ endMeetingNotificationList = ["jose"]
10
11 #TODO: endMeetingMessage should get filenames from the writers
12
13
14=== modified file 'writers.py'
15--- writers.py 2014-02-07 16:13:27 +0000
16+++ writers.py 2014-02-07 17:18:24 +0000
17@@ -1218,7 +1218,7 @@
18 if len(M.votes) > 0:
19 for m in M.votes:
20 # differentiate denied votes somehow, strikethrough perhaps?
21- Votes.append(" * [[%(fullLogsFullURL)s#"+str(M.votes[m][3])+" "+m+"]]")
22+ Votes.insert(0," * [[%(fullLogsFullURL)s#"+str(M.votes[m][3])+" "+m+"]]"))
23 motion = "Deadlock"
24 if(M.votes[m][0] > M.votes[m][1]):
25 motion = "Motion carried"
26@@ -1227,7 +1227,7 @@
27 Votes.insert(0," * " + motion + " (For/Against/Abstained "+str(M.votes[m][0])+"/"+str(M.votes[m][2])+"/"+str(M.votes[m][1]) + ")")
28 if len(M.publicVoters[m]) > 0:
29 publicVoters = ', '.join(set(M.publicVoters[m]))
30- Votes.append(" * Voters " + publicVoters)
31+ Votes.insert(0, " * Voters " + publicVoters)
32 Votes = "\n".join(Votes)
33 return Votes
34

Subscribers

People subscribed via source and target branches