Merge lp:~cjohnston/ubuntu-classroom/un-voice-admins into lp:~ubuntu-classroom/ubuntu-classroom/classbot

Proposed by Chris Johnston
Status: Merged
Approved by: Nathan Handler
Approved revision: not available
Merge reported by: Nathan Handler
Merged at revision: not available
Proposed branch: lp:~cjohnston/ubuntu-classroom/un-voice-admins
Merge into: lp:~ubuntu-classroom/ubuntu-classroom/classbot
Diff against target: 14 lines (+0/-4)
1 file modified
classbot.pl (+0/-4)
To merge this branch: bzr merge lp:~cjohnston/ubuntu-classroom/un-voice-admins
Reviewer Review Type Date Requested Status
Nathan Handler Approve
Review via email: mp+19777@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Chris Johnston (cjohnston) wrote :

Removes auto +v from admins

Revision history for this message
Nathan Handler (nhandler) wrote :

I guess this makes sense. Admins are not really part of the session, and they can use !voice if they need to gain voice to talk.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'classbot.pl'
2--- classbot.pl 2010-02-20 15:01:16 +0000
3+++ classbot.pl 2010-02-20 16:06:15 +0000
4@@ -123,10 +123,6 @@
5 &voice($chan, $key);
6 &voice($qchan, $key);
7 }
8- for my $key (keys %admins) {
9- &voice($chan, $key);
10- &voice($qchan, $key);
11- }
12 for my $key (keys %shelpers) {
13 &voice($chan, $key);
14 &voice($qchan, $key);

Subscribers

People subscribed via source and target branches

to all changes: