Background:
This issue is part of the task to update the hook_help texts of the Drupal 8 modules:
#1908570: [meta] Update or create hook_help() texts for D8 core modules

Tasks:
- review / write the hook_help text according to help guidelines

Files: 
CommentFileSizeAuthor
#5 interdiff.txt2.04 KBbatigolix
#5 update-hook-help-for-rdf-module-2091357-5.patch2.1 KBbatigolix
PASSED: [[SimpleTest]]: [MySQL] 60,108 pass(es). View
#3 interdiff.txt2.03 KBbatigolix
#3 update-hook-help-for-rdf-module-2091357-3.patch2.1 KBbatigolix
PASSED: [[SimpleTest]]: [MySQL] 59,537 pass(es). View
#1 update-hook-help-for-rdf-module-2091357-1.patch2.09 KBbatigolix
PASSED: [[SimpleTest]]: [MySQL] 58,613 pass(es). View

Comments

batigolix’s picture

Component: documentation » rdf.module
Status: Active » Needs review
FileSize
2.09 KB
PASSED: [[SimpleTest]]: [MySQL] 58,613 pass(es). View

patch:

- changes @tokens to !tokens
- changes http to https link for d.o.

also changing the component in case the rdf maintainers would like to comment or know about any recent UI changes

jhodgdon’s picture

Status: Needs review » Needs work

Could we also update the "for more information" section? It still says "handbook" and does not conform to the template. Thanks!

batigolix’s picture

Issue summary: View changes
Status: Needs work » Needs review
FileSize
2.1 KB
PASSED: [[SimpleTest]]: [MySQL] 59,537 pass(es). View
2.03 KB

You're welcome

The links work

jhodgdon’s picture

Status: Needs review » Needs work

Nearly perfect! There is a "which" there that needs a comma before it though:

...RDFa specification which allows RDF data to be embedded in HTML markup.

batigolix’s picture

Status: Needs work » Needs review
FileSize
2.1 KB
PASSED: [[SimpleTest]]: [MySQL] 60,108 pass(es). View
2.04 KB

New patch, with comma

batigolix’s picture

jhodgdon’s picture

Component: rdf.module » documentation
Status: Needs review » Reviewed & tested by the community

Looks good! I moved this back to documentation component, since we are not changing the substance of the existing help that the module maintainers put in.

This is ready to commit. I don't think it needs a manual test even.

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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