Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#1781 closed enhancement (fixed)

Explanation Editor role

Reported by: drewapicture's profile DrewAPicture Owned by: coffee2code's profile coffee2code
Milestone: Priority: normal
Component: Developer Hub Keywords: has-patch


Following the Codex Migration status update I posted on make/docs this morning, I think we're finally to a point where more contributors can selectively be given access to the DevHub back-end for migrating curated documentation from the Codex.

That attached patch adds the new role and grants explanation caps to admins, editors, and explanation editors. Explanation editors will still not be able to push explanations live, but they'll certainly be able to create new ones and submit them for review, as well as edit published explanations.

Attachments (2)

1781.diff (11.0 KB) - added by DrewAPicture 8 years ago.
1781.2.diff (11.0 KB) - added by DrewAPicture 8 years ago.

Download all attachments as: .zip

Change History (8)

8 years ago

8 years ago

This ticket was mentioned in Slack in #meta-devhub by drew. View the logs.

8 years ago

#2 @coffee2code
8 years ago

In 3607: Add explanation editor role.

Props DrewAPicture.
See #1781.

#3 @coffee2code
8 years ago

  • Owner set to coffee2code
  • Resolution set to fixed
  • Status changed from new to closed

In 3608: Expose the explanations post type for administration.

  • Allow post type to appear in admin menu.
  • Assign menu icon.
  • Prevent "Explanations" from being listed in the admin bar "New" menu.
  • Remove "Add New" from admin submenu.
  • Remove "Add New" from head of "Explanations" admin pages.
  • Add count of pending explanations to admin menu.
  • Prevent access to the admin page for creating a new explanation.

Fixes #1781.

#4 @DrewAPicture
8 years ago

@coffee2code I think you still need to manually fire the after_switch_theme action so the new role gets added. It's not showing up in the drop-down because it doesn't exist yet :-)

#5 @drewapicture
8 years ago

In 3661: Check if the user can edit an explanation before giving them an edit link in the Toolbar.

See #1534. See #1781.

#6 @coffee2code
8 years ago

@DrewAPicture: Ah, I did indeed neglect to do so. It's done now. :)

Note: See TracTickets for help on using tickets.