Closed (fixed)
Project:
User Guide
Version:
8.x-3.0
Component:
User Guide content
Priority:
Minor
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
30 Aug 2017 at 10:17 UTC
Updated:
18 Oct 2017 at 23:09 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
jhodgdonThanks, we should definitely fix that!
Comment #3
eojthebraveI found two instances of "Basic page" being used instead of "Vendor" in this page. Here's a patch that updates both of them. I also walked through the steps to make sure those labels are indeed correct. And they are.
Comment #4
jhodgdonPatch looks good to me, thanks!
I suggest we commit all the content patches that are being worked on together, and put out one email notification to the translation groups, rather than one notification. There's a template for the email in the Maintainers guide... Let's wait until after the sprint and put out the notification then? I can take care of it Monday.
Comment #5
eojthebrave+1 to doing all the content edits at once. Makes things easier.
- #2912963: A minimum disk space in case of minimum installation (3.1. Concept: Server Requirements).
- #2906889: Table of form field values missing on config-basic
- #2895649: Place block in region no longer includes - None -
- and maybe #2900226: Tables of fields/values are inconsistent regarding fieldsets
All seem like candidates.
Comment #6
jhodgdonAlso #2912518: Relevance of firebug in D8 Documentation (which is ready to go, RTBC patch).
The one about inconsistency #2900226: Tables of fields/values are inconsistent regarding fieldsets we should probably do separately. It will likely itself touch a number of topics, and it hasn't been fully analyzed yet.
But the other three listed in #5 need patches. Should be fairly quick and straightforward... I can probably take them on in the next few days.
Comment #7
jhodgdonI've created proposed patches for those 3 issues, ready for review/revision!
Comment #8
jhodgdonOne more patch to review in this batch: #2888863: Enabling & Disabling Maintenance Mode (extend-maintenance) should mention Drush
Comment #9
jhodgdonComment #11
jhodgdonI committed this patch, thanks again! Sending out translation notification shortly, along with several other issues. There are a few more coming in a few days but I thought I'd get a batch out today...