Moderation is not available anymore.

CommentFileSizeAuthor
#8 help_11.patch4.56 KBGurpartap Singh
#6 help_10.patch4.94 KBRobRoy
#3 help_8.patch3.79 KBkkaefer
#2 help_7.patch3.16 KBkkaefer
remove-moderation.patch1.3 KBkkaefer
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kkaefer’s picture

kkaefer CreditAttribution: kkaefer commented
Category: task » bug

It's actually a bug

kkaefer’s picture

kkaefer CreditAttribution: kkaefer commented
FileSize
help_7.patch3.16 KB

Changed some strings as well to account recent changes (i.e. blocks are not longer restricted to sidebars).

kkaefer’s picture

kkaefer CreditAttribution: kkaefer commented
Title: Remove information about moderation from help page » Clean up help.module front page
Version: x.y.z » 5.x-dev
FileSize
help_8.patch3.79 KB

Updated.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Title: Clean up help.module front page » String freeze: Clean up help.module front page

Patch that impacts the string freeze. Changing status to critical task. Will try and review tomorrow when I am more awake.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Category: bug » task
Priority: Normal » Critical
RobRoy’s picture

RobRoy CreditAttribution: RobRoy commented
FileSize
help_10.patch4.94 KB

The previous patch introduced some typos and funky wording. This improves those typos and improves the help text.

drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm commented
Status: Needs review » Needs work

I think 'Further' can be left out.

<p> should go outside t() on the last part (the one that is only one sentence).

Gurpartap Singh’s picture

Gurpartap Singh CreditAttribution: Gurpartap Singh commented
Status: Needs work » Needs review
FileSize
help_11.patch4.56 KB

<p> doesn't need to go out of t(), it's multi-line text. Also, restored the "Futher ..." line. Rest is same as appears in above patches.

drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm commented
Status: Needs review » Active

Committed to HEAD.

'node' should be replaced with 'post' and/or 'content.'

kkaefer’s picture

kkaefer CreditAttribution: kkaefer commented

I don’t think that ‘node’ should be replaced with something else. Users might see that term somewhere and will look it up on the help page.

drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm commented

Any UI text that mentions 'node' is a bug. node should strictly be an API term.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented

Er. But this is a glossary? Surely it makes sense to mention it here? There's help for "Node" module listed at the top; it's reasonable to assume that someone might see that and wonder, "What's a node?" What better place to find the answer to this question than in the glossary?

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented

Further, "node" is in all content URLs, which is as "user-facing text" as you can get. Again, it makes sense to have a definition here.

drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm commented
Status: Active » Closed (fixed)

Okay.