We need individual rights for locales

RESOLVED FIXED in 2013Q2

Status

support.mozilla.org
Knowledge Base Software
P3
normal
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: atopal, Assigned: rrosario)

Tracking

unspecified
2013Q2

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: u=contributor c=wiki p=3 s=2013.10)

(Reporter)

Description

5 years ago
Currently we can assign rights to delete articles broken down by locales. We also need the same for reviewing revisions.
(Assignee)

Updated

5 years ago
Whiteboard: u=contributor c=wiki s=2012.3 p= → u=contributor c=wiki s=2012.4 p=
(Assignee)

Updated

5 years ago
Whiteboard: u=contributor c=wiki s=2012.4 p= → u=contributor c=wiki s=2012.4 p=1
(Assignee)

Comment 1

5 years ago
James just convinced me that this is more like a 2pt bug. I was thinking just a migration with a bunch of inserts but then thinking through, we have to touch the views and templates to check these templates. And then add tests to make sure we are doing it right.
Whiteboard: u=contributor c=wiki s=2012.4 p=1 → u=contributor c=wiki s=2012.4 p=2
(Reporter)

Updated

5 years ago
Priority: -- → P3
(Reporter)

Comment 2

5 years ago
After talking with Michael it seems like we can get away with just making English a special case. So we'd have reviewer rights for English and reviewer rights for other locales. We'd also need the same for can_delete_revision
Summary: We need reviewing rights broken down by locale → We need reviewing rights for en-US and other locales
(Assignee)

Comment 3

5 years ago
Didn't make it in 2012.4.
Whiteboard: u=contributor c=wiki s=2012.4 p=2 → u=contributor c=wiki s=2012.5 p=2
Bumping to 2012.6.
Whiteboard: u=contributor c=wiki s=2012.5 p=2 → u=contributor c=wiki s=2012.6 p=2
(Assignee)

Comment 5

5 years ago
Sprint based milestone :-)
Target Milestone: --- → 2012.6
(Assignee)

Updated

5 years ago
Whiteboard: u=contributor c=wiki s=2012.6 p=2 → u=contributor c=wiki p=2
(Assignee)

Updated

5 years ago
Target Milestone: 2012.6 → 2012.7
(Assignee)

Updated

5 years ago
Target Milestone: 2012.7 → 2012.8
(Reporter)

Updated

5 years ago
Target Milestone: 2012.8 → 2012.9
(Reporter)

Updated

5 years ago
Target Milestone: 2012.9 → 2012.10
(In reply to Ricky Rosario [:rrosario, :r1cky] from comment #1)
> James just convinced me that this is more like a 2pt bug. I was thinking
> just a migration with a bunch of inserts but then thinking through, we have
> to touch the views and templates to check these templates. And then add
> tests to make sure we are doing it right.

That sounds like 3 points of work--not 2. Can this be done in a day or two?
(Reporter)

Updated

5 years ago
Target Milestone: 2012.10 → 2012.11
(Assignee)

Updated

5 years ago
Target Milestone: 2012.11 → 2012Q3

Comment 7

4 years ago
Related discussion:

https://support.mozilla.org/en-US/forums/contributors/709054?#post-51384
(Assignee)

Comment 8

4 years ago
We created the KB locale teams models to handle this. Each active locale in: https://support.mozilla.org/en-US/kb/locales needs to have it's leaders and reviewers assigned and then we can switch to using those roles for the actual permissions.
(Reporter)

Comment 9

4 years ago
I removed the points since the base for this changed and made this bug more generic. If it's too much, let's make this a tracker bug and file more, but for now, this is what we'd need:

* delete revision
* review revision
* change keywords
Summary: We need reviewing rights for en-US and other locales → We need individual rights for locales
Whiteboard: u=contributor c=wiki p=2 → u=contributor c=wiki p= s=2013.8
Target Milestone: 2012Q3 → 2013Q2
(Assignee)

Comment 10

4 years ago
We need to be 100% sure all the locale teams are setup before doing this. Otherwise, all hell will break lose. Moving to backlog.
Whiteboard: u=contributor c=wiki p= s=2013.8 → u=contributor c=wiki p= s=2013.backlog
(Reporter)

Comment 11

4 years ago
I'm moving this to the 9th sprint after talking to Rosana. She said she'd have that ready within two weeks.
Whiteboard: u=contributor c=wiki p= s=2013.backlog → u=contributor c=wiki p= s=2013.9

Comment 12

4 years ago
I will work on moving all the reviewers to their locales. Since this is a manual process it will take some time, but I will be ready before the next sprint.
(Assignee)

Comment 13

4 years ago
Are the locales all set for this to be implemented?

What are the actual permissions per locale that each role gets you? Is it just:

Locale Leaders:
* Allowed to delete documents in their locale

Locale Reviewers:
* Allowed to delete revisions in their locale
* Allowed to review revisions in their locale
Flags: needinfo?(rardila)
(Assignee)

Comment 14

4 years ago
Making this a 3pter (conservatively). Might end up being 2pt. Might end up needing followup bugs.
Whiteboard: u=contributor c=wiki p= s=2013.9 → u=contributor c=wiki p=3 s=2013.9
(Assignee)

Comment 15

4 years ago
Moving to backlog because I don't see locales set up. For example:
https://support.mozilla.org/en-US/kb/locales/ro
Whiteboard: u=contributor c=wiki p=3 s=2013.9 → u=contributor c=wiki p=3 s=2013.backlog
(Reporter)

Comment 16

4 years ago
Moving this into the next sprint. Rosana said on IRC she will have it ready by May 14th. What we have right now is that locale leaders and reviewers have exactly the same rights, but your suggestions is correct.

To the ones you mentioned, please also add:
* Can change document (for keywords)

Bot for reviewers (and of course locale leaders get everything that reviewers have)
Whiteboard: u=contributor c=wiki p=3 s=2013.backlog → u=contributor c=wiki p=3 s=2013.10
(Assignee)

Comment 17

4 years ago
(In reply to Kadir Topal [:atopal] from comment #16)
> To the ones you mentioned, please also add:
> * Can change document (for keywords)


keywords are still revision level and available to edit by all users. 

See bug 860225
(Reporter)

Comment 18

4 years ago
Yeah, was thinking of limiting that to reviewers too, like we do in English, but now that I think about it, maybe there are more down than upsides to that. Let's skip that for now.
(Assignee)

Comment 19

4 years ago
This is next for me.
Assignee: nobody → rrosario
(Assignee)

Updated

4 years ago
Flags: needinfo?(rardila)
(Assignee)

Comment 20

4 years ago
Pull request is in action here:
https://github.com/mozilla/kitsune/pull/1376
(Assignee)

Comment 21

4 years ago
Landed on master:
https://github.com/mozilla/kitsune/commit/8cd1303d8fc459bf5ca9afbf5e73e727982ce88a
(Assignee)

Comment 22

4 years ago
Deployed to prod by mythmon now.
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
(Assignee)

Comment 23

4 years ago
To summarize the current state of wiki permissions:

Archive Document
* Users with the wiki.archive_document can archive documents

Make Ready for L10n
* This only applies to localizable en-US documents
* Only users with wiki.mark_ready_for_l10n permission can mark a document as ready for l10n

Edit Keywords:
* All logged in users can edit (translate) keywords for non-en-US documents
* Only users with wiki.edit_keywords permission can edit keywords for en-US documents

Edit Needs Change:
* This only applies to en-US documents
* Users with wiki.edit_needs_change permission can edit needs change fields

Edit Other Document Fields
* Any logged in user can edit the other document/descriptio  fields (title, slug, ...) until there is an approved revision. After that...
* Locale leaders and reviewers can edit for documents in there locale
* All users with the wiki.change_document permission can edit for documents in all locales

Delete Document
* Locale leaders can delete documents in their locale
* Users with the wiki.delete_document can delete documents in any locale

Create Revision
* Any logged in user can create a revision in any locale

Review Revision
* Locale leaders and reviewers can review revisions for documents in their locale
* Users with wiki.delete_revision permission can review revisions for documents in all locales

Delete Revision
* Locale leaders and reviewers can delete revisions for documents in their locale
* Users with wiki.review_revision permission can delete revisions for documents in all locales


whewwww
You need to log in before you can comment on or make changes to this bug.