Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
Follow-up to #2899553: Architectural review of the Workflows module (documentation cleanups). WorkflowListBuilder
.1.
Comments
Comment #2
Wim LeersComment #3
Sam152 CreditAttribution: Sam152 as a volunteer and at PreviousNext commentedComment #4
karthikkumarbodu CreditAttribution: karthikkumarbodu as a volunteer and at Axelerant commentedComment #5
timmillwoodI'm surprised changing this text didn't cause a test fail, so maybe we need a test here.
Comment #6
karthikkumarbodu CreditAttribution: karthikkumarbodu as a volunteer and at Axelerant commentedAdded Tests to verify if Content Moderation text is available as a link.
Comment #7
timmillwoodNice work
Comment #8
xjmThis is a good UX improvement; I was wondering about this myself. Normally, we don't put any references to experimental modules in stable modules, but Content Moderation and Workflows are kind of a special case. And the user will get the necessary warnings about the beta module when they look into it.
There should not be spaces around " Content Moderation " inside the
<a>
tag; just the space in the middle.Comment #9
karthikkumarbodu CreditAttribution: karthikkumarbodu as a volunteer and at Axelerant commentedRemoved space in the anchor tag text "Content Moderation"
Comment #10
Sam152 CreditAttribution: Sam152 as a volunteer and at PreviousNext commentedLooks good.
Comment #12
xjmAlright, thanks! Committed to 8.5.x. Since 8.4.x is in RC, per the D8 allowed changes policy, string changes should be committed to the next minor version, so I did not backport it. (This does not apply to experimental modules, but Workflows is now marked stable.) :)
Comment #13
xjm