See #69556: move "cvs access" tab into project, make it generic for background. Once that lands, we need a patch to add a 'Maintain issues' checkbox at node/N/maintainers and use that to populate the available options for the Assigned dropdown.

Comments

dww’s picture

Status: Active » Needs review
StatusFileSize
new7.83 KB
mikey_p’s picture

Seems like ideally there'd be a separate permission for 'administer project issues' allowing them access to node/$pid/edit/issues?

Edit: While this may be true, there are some other sticky issues here with controlling access to the primary Edit tab and whether or not it would be visible if the project edit tab is not visible...might have to try this out and see what happens.

mikey_p’s picture

Here's an example of what I was trying to describe:

Only local images are allowed.

dww’s picture

StatusFileSize
new6.7 KB

Rerolled #1 based on the latest API changes over at #69556-30: move "cvs access" tab into project, make it generic.

mikey_p and I talked about an 'administer project issues' perm in IRC. Summary: let's not add that unless we actually need it. For now, we'll just say you have to be able to edit the project node to change those issue-related settings. It's cool we've got a flexible system now, but let's not go crazy making it extra granular if there's no real need for it.

dww’s picture

Re: #3: No, that tab won't be visible. But, the direct links to those pages still work. Try it yourself. ;) If you give a user just 'administer project releases' perm, but not 'administer project' perm, the "Administer releases" link on the project page, which sends you to node/N/edit/releases still works and the page renders. And you see the "View" tab. But no Edit tab and no subtabs for issues and releases.

dww’s picture

dww’s picture

Both sun and wonder95 don't like "Maintain issues", and both proposed "Assign issues" instead. Anyone can still assign issues to themselves. So, it's a bit misleading. It really means "Assign issues to anyone else with this permission". That's why I thought "Maintain issues" was good -- you're an issue maintainer. And, it's possible we'd dream up some other powers we'd give to people with this. But, I'm open to a better suggestion if anyone's got one.

sun’s picture

Hm. "Assign issues to anyone else with this permission" actually is what I'd understand for "Assign issues" - otherwise, I'd expect the label "Be assigned to issues" :)

dww’s picture

Bojhan reviewed and thought "Maintain issues" was actually fine for this, so I'm going to leave it like that for now.

dww’s picture

Status: Needs review » Fixed

Committed, merged into bzr, deployed on d.o. DB updates ran without error. Hurray!

Status: Fixed » Closed (fixed)
Issue tags: -drupal.org redesign, -drupal.org redesign project, -drupal.org redesign sprint 2

Automatically closed -- issue fixed for 2 weeks with no activity.