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

Proposed by Thomas Molloy
Status: Merged
Merged at revision: 15
Proposed branch: lp://staging/~lderan/ubuntu-bots/meeetingology-output
Merge into: lp://staging/~ubuntu-bots/ubuntu-bots/meetingology
Diff against target: 277 lines (+76/-16)
4 files modified
items.py (+3/-0)
meeting.py (+42/-10)
plugin.py (+3/-0)
writers.py (+28/-6)
To merge this branch: bzr merge lp://staging/~lderan/ubuntu-bots/meeetingology-output
Reviewer Review Type Date Requested Status
Alan Bell Approve
Review via email: mp+195883@code.staging.launchpad.net

Description of the change

Commands that are not publicly useable are privately sent to the chairs, and when a new chair gets added they are sent the list as well. This does result in a list that will have to be maintained if new public commands are added.

DONE meeting item added.

Meeting notifications when they end sent to preset irc nicks.

Votes now link to the HTML log url to the point where the voting begins in the MoinMoin output.

Votes are now in chronological order in the MoinMoin output.

To post a comment you must log in.
Revision history for this message
Alan Bell (alanbell) wrote :

good stuff

review: Approve
15. By Alan Bell

accepting merge

Revision history for this message
Alan Bell (alanbell) wrote :

ooh hang on, not sure that the hard coded lderan notification nick is a good idea

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches