Closed (outdated)
Project:
Drupal core
Version:
8.2.x-dev
Component:
content_moderation.module
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
22 Sep 2016 at 20:34 UTC
Updated:
8 Nov 2016 at 13:05 UTC
Jump to comment: Most recent, Most recent file


Comments
Comment #2
jhedstromComment #3
pericxc commentedComment #5
pericxc commentedComment #6
jhedstromIt would be good to get some of the other folks that helped review #2750365: Permissions: moderation states with the same name is unclear for user to assign to a role to chime in here as well.
Some code feedback.
This is off a bit since this trait deals with unique and non-unique. Perhaps "Provides a trait for getting labels."
I'm not sure traits can use
{@inheritdoc}...Tiny nitpick, how about something more specific than
$arg?Let's add code comments clarifying why we don't need a unique label here.
And here too.
Comment #7
pericxc commentedComment #8
jhedstromI think since displaying the machine name can be seen as unfriendly, perhaps this patch can be updated to only display the machine name if the site in question has duplicate labels?
Comment #9
pericxc commentedComment #11
pericxc commentedComment #12
pericxc commentedComment #14
pericxc commentedComment #16
pericxc commentedComment #18
pericxc commentedComment #20
lomasr commentedApplied the patch #5 worked cleanly but cant see the machine name added. Do I need to Add moderation. Please see screenshot.
Comment #21
pericxc commentedhi @lomasr machine_name will appear ONLY for duplicate labels. In order to see the results, create one more Draft state ( or published).
Comment #22
pericxc commentedComment #23
alexpottThis is solved by #2779647: Add a workflow component, ui module, and implement it in content moderation
Comment #24
sam152 commentedBetter status given the issue linked in #23?