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.
Background:
This issue is part of the meta-issue for updating 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 http://drupal.org/node/632280
More details are on the meta-issue.
Comment | File | Size | Author |
---|---|---|---|
#20 | hook_help_user_module-2103043-20.patch | 6.18 KB | JiriK |
#20 | interdiff-2103043-15-20.txt | 4.58 KB | JiriK |
#16 | hook_help_user_module-2103043-15.patch | 6.17 KB | batigolix |
#16 | interdiff.txt | 4.74 KB | batigolix |
#12 | interdiff.txt | 6.36 KB | batigolix |
Comments
Comment #1
jhodgdoncomponent
Comment #2
Sree CreditAttribution: Sree commentedMissing doc on many updated features ... update is needed.
Assigning ....
Also, should I update the corresponding doc page on drupal.org for drupal8 with the updated details?
Comment #3
Sree CreditAttribution: Sree commentedUpdated about default roles
Comment #4
Sree CreditAttribution: Sree commentedUpdated default roles, bulk operations
Comment #5
jhodgdonThe patch file in #4 has two sections for the user module... weird. Can you make a new patch with just one copy of your intended changes? I wasn't sure which section to review.
Comment #6
Sree CreditAttribution: Sree commentedThere was some weird issue with my git ... here is the latest patch which has updated help on user roles
Comment #7
ifrikThanks for the patch. I'm still catching up from the Sprint in Prague...
Some comments:
1) User roles and permissions are explained in detail as an item under issues. Therefore it can be shorter in the About section, and we can get rid of the bracktes that way. May something like as short as
"It also allows users with appropriate permissions to manage user roles and permissions."
2) The link to the online documentation should follow the wording in the Help Text Standard: https://drupal.org/node/632280
3) "Creating users": Maybe instead of saying "If allowed" - add a link to the Account settings page, and phrase it something like:
If the registration settings on the <link>Account settings page</link> allow vistors to create their own account, then ...
4) User roles: There are now is "administrator" as a third role. How about: "Typically there are three pre-defined roles". Explaining them, and then just explain how to add new roles, and that you then can grant additional permissions.
The roles page is not actually in the admin menu, so maybe we need to say that this is part of the People page?
5) Maybe split user roles and permissions, because that's getting a bit long? It needs mentioning that any new role automatically has all the permissions of the authenticated user.
6) Account settings: Maybe the whole issue of Email texts should be moved to a seperate item? The emails are not only for the registration, but also for requesting new passwords, or when an account gets blocked or deleted. Plus the use of the variables in the text might need mentioning.
7) The links need to be changed according to the new routing format, as explained on at https://drupal.org/node/632280
Comment #8
jhodgdonstatus update as per #7
Comment #9
Sree CreditAttribution: Sree commented#7: Thanks for the detailed review with comments. I ll work on the suggested updates & would submit another patch.
Comment #10
Sree CreditAttribution: Sree commentedComment #11
batigolixComment #12
batigolixThe attached patch addresses point 1) to 7) of comment #7
I didnt split up the Account settings section as suggested in 6) of #7
I added a section about managing User fields , as this was missing.
It is a bit experimental as this is the first entity type that mentions this option (?)
So please review in such way that it can be re-used in hook_helps for taxonomy, custom block etc
Comment #13
jhodgdonThanks, this is looking pretty good!
There are a few problems with this patch:
a) In the first Uses bullet, there is a URL that isn't actually mentioned in the text and can be removed (@account-settings).
b) In the first Uses bullet, please avoid saying "his". Make it more generic, like "the".
c) Drupal text standards require serial commas, so "user name, account blah blah blah, and password" (requires comma before "and").
d) Unfortunately our standard requires we write "e-mail" not "email". Also, "e-mails" is incorrect, as "mail" and "e-mail" are not countable nouns. Say "e-mail messages".
e) Punctuation:
"By adding fields for e.g., a picture, a biography or address you can a create a custom profile for the users of the website."
should be
"By adding fields for e.g., a picture, a biography, or address, you can a create a custom profile for the users of the website."
f) I think a better heading for "Managing user profile" would be "Managing user fields". I think the text there is pretty good, but it should probably say that user accounts are "an entity type" rather than being plural "entity types". The other modules that allow fields, such as Node, just link to the field module and don't say much, but it's a lot more complicated there because the fields are on individual content types.
Comment #15
batigolixPatch adresses points a) - f) from #13
I used "Managing user account fields" isntead of the suggested "Managing user fields" as a header
Comment #16
batigolixPatch addresses a) - f) from #13
For f) I chose "Managing user account fields" as a header
Comment #17
jhodgdonI think this is great! We probably need someone to manually test it:
- Make sure all the links work
- Make sure the formatting is OK
- Verify that wherever specific pages, buttons, tab titles, etc. are mentioned, they match what you would see in the user interface.
Comment #18
batigolixComment #19
jhodgdonI did a manual test of this patch today. It all looks great except a few typographical issues:
a) Uses / Creating and Managing - paragraph does not end in ".".
b) Uses / User roles - the three roles that are set up for me by the Standard profile are actually capitalized. "Anonymous user", "Authenticated user", and "Administrator".
c) Same paragraph: needs a comma before "and Administrator".
d) Uses / Account settings:
"...whenever a module is enabled.), personal contact forms..." ==> no . before the ) here
Comment #20
JiriK CreditAttribution: JiriK commentedMy first novice try, hope I did it correctly...
Patch addresses a) - d) from #19
Comment #21
jhodgdonGreat, thanks!
Comment #22
JurgenR CreditAttribution: JurgenR commentedNice work.
Comment #23
jhodgdonA patch on an issue tagged "avoid commit conflicts" touches user.module: #1996238: Replace hook_library_info() by *.libraries.yml file
So I'm being extra-careful and will wait to commit this until that other issue is finished.
Comment #24
jhodgdonI've been given leave from sun (the owner of that other issue/patch) to go ahead with commits on this file, so I've committed this to 8.x.
Thanks everyone!