413541 - Create group for locale leaders

Status: VERIFIED FIXED

(support.mozilla.org :: General, defect)

Description

[Chris Ilias [:cilias]]

We need to create a user group on Firefox Support specifically for locale leaders; just in case we need to give them special permissions different from other approvers.

Comment 1

[Jason Barnabe (np)]

I created a "Locale leaders" group with the same permissions as "Approvers" and assigned my test account to it. I didn't have the ability to approve things. Maybe that sort of thing is hard coded somewhere?

To get things rolling, let's put locale leaders as approvers. Once we figure out what the problem is, we'll change them.

Comment 2

[David Tenser [:djst]]

(In reply to comment #1)
> I created a "Locale leaders" group with the same permissions as "Approvers" and
> assigned my test account to it. I didn't have the ability to approve things.
> Maybe that sort of thing is hard coded somewhere?
> 

Nelson, do you know what's up with that?

> To get things rolling, let's put locale leaders as approvers. Once we figure
> out what the problem is, we'll change them.
> 

Sounds good. 

Comment 3

[Chris Ilias [:cilias]]

This group needs the ability to remove articles.

Comment 4

[Jason Barnabe (np)]

Which group, the one that I tried to make but it didn't work or "Approvers"?

Comment 5

[Nelson Ko [:nkoth]]

this sort of thing should have no hard-coding. Jason, your new group should work. However, note the "included permissions" through the "include" feature,

Comment 6

[Chris Ilias [:cilias]]

Jason, I'm referring to the locale leaders group.

Maybe I don't understand the way inherited permissions work, but do you think we should make the locale leaders group inherit all permissions from Approvers, then we can add any additional permissions?

Comment 7

[Nelson Ko [:nkoth]]

You will notice that "Registered" includes "Approvers". This means "Approvers" has permissions of "Registered" plus any additional set for itself.

I think it is a good idea to simply set "Locale leaders" to include "Approvers", and then add additional perm if necessary. This is more hierarchical and manage-able.

Comment 8

[Jason Barnabe (np)]

(In reply to comment #5)
> Jason, your new group should
> work. However, note the "included permissions" through the "include" feature,

Should, but doesn't. Try it out.

We can make "Locale leaders" inherit from "Approvers" as long as we're not going to be *removing* permissions that Approvers would ordinarily have.

Comment 9

[Nelson Ko [:nkoth]]

I think this group may not have the necessary category permissions (tiki-admin_categories.php) to do what it needs to do to objects within a certain category? (But the include/inherit feature should give it the category perms too). Does the include/inherit work?

Comment 10

[Jason Barnabe (np)]

Yeah, that's probably the problem. I'll try it out once I get a decision about whether we really want the group to inherit from Approvers. Chris?

Comment 11

[Chris Ilias [:cilias]]

I can't think of a possible permission we would want approvers to have and locale leaders not to have. Make Locale Leaders inherit Approvers permissions.

Comment 12

[Jason Barnabe (np)]

I can't think of a permission that we'd want to add either, but OK. I've made the group, it seems to work. Find a guinea pig and give them the group, and if everything's fine switch over all of them.

Comment 13

[Chris Ilias [:cilias]]

I used Chris_Ilias_user as test user. The features are available.