946830 - Ability to set up discussions from request wizard

Status: VERIFIED FIXED

(Webtools Graveyard :: Air Mozilla, defect, P1)

Description

[Peter Bengtsson [:peterbe]]

Note: This bug depends on use figuring out exactly the policies around moderation and notifications etc. Most likely it'll be a matter of just checking a box "Enable discussion on this event" and an additional input to add more moderators other than yourself.

Comment 2

[Peter Bengtsson [:peterbe]]

We discussed this briefly and I *think* the right course of action is this::

1. On the Details page we ask "Do you want to allow discussions?" Enabling this will expose two more form-widgets. 

2. A checkbox on there for "Moderate all comments?" which is checked "by force" if the event is NOT private. Meaning all discussions have to be moderated if it's not MoCo private. 

3. A textarea with the label "People to act as moderators?" and already it will have the requester's email entered there already. 

If I remember correctly, if you're a moderator of a discussion you ALWAYS get notifications. I think that's good. 

What do you think?

Comment 3

[Richard A Milewski[:richard]]

I think you can turn off notifications all together.  But maybe that's just from the admin interface.

Not sure about #2.  ...but check my logic, I may be wrong:

Only people who are logged-in can commment.  That means they are staff or mozillians.  So I think the risk of allowing unmoderated comments is lower than most sites.   I think we should be as open as possible and tighten things up if we find people behaving badly.

Thoughts?

Comment 4

[David Weir (satdav)]

Will we be able to add multi moderators if so will this need to be done by a bug

Comment 5

[Peter Bengtsson [:peterbe]]

We talked about this in the meeting. I think we basically concluded that my plan in Comment 2 is solid.

Comment 6

[Richard A Milewski[:richard]]

I agree!

Comment 7

[Peter Bengtsson [:peterbe]]

Should this be enabled by default? 
I'm inclined to say yes. Stop me before it's too late :)

Comment 8

[Peter Bengtsson [:peterbe]]

Attachment: Screenshot 2014-03-11 16.57.16.png

Good progress today. 

I've made it so that discussion is enabled by default for all new requests.
The "Moderate all" is always on if the event isn't private. 

The list of email address is the list of email addresses that always gets notified with or without the need to moderate.

Comment 9

[Peter Bengtsson [:peterbe]]

This is about to land on dev. 

I am, for this time, setting discussion to be enabled by default. If you think that's a bad idea, we can easily disable it. 

Please take the time to verify this bug once it's resolved.

Comment 10

[[github robot]]

Commit pushed to master at https://github.com/mozilla/airmozilla

https://github.com/mozilla/airmozilla/commit/9cfbf2a3160259b72e0b375af0c738b84e1c84c2
fixes bug 946830 - Ability to set up discussions from request wizard

Comment 11

[Richard A Milewski[:richard]]

Looks good.   One VERY minor nit.  We might want to change the phrasing from "...have to have the moderation..." to "...must have the moderation..."