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
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | interdiff.txt | 2.04 KB | batigolix |
| #5 | update-hook-help-for-rdf-module-2091357-5.patch | 2.1 KB | batigolix |
| #3 | interdiff.txt | 2.03 KB | batigolix |
| #3 | update-hook-help-for-rdf-module-2091357-3.patch | 2.1 KB | batigolix |
| #1 | update-hook-help-for-rdf-module-2091357-1.patch | 2.09 KB | batigolix |
Comments
Comment #1
batigolixpatch:
- 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
Comment #2
jhodgdonCould we also update the "for more information" section? It still says "handbook" and does not conform to the template. Thanks!
Comment #3
batigolixYou're welcome
The links work
Comment #4
jhodgdonNearly 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.
Comment #5
batigolixNew patch, with comma
Comment #6
batigolixComment #7
jhodgdonLooks 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.
Comment #8
webchickCommitted and pushed to 8.x. Thanks!