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: http://drupal.org/node/537828
Comment | File | Size | Author |
---|---|---|---|
#9 | help_openid3.png | 107.38 KB | arianek |
#9 | help_openid3.patch | 5.09 KB | arianek |
#8 | openid_help_2.patch | 4.14 KB | batigolix |
#7 | openid_help_2.patch | 5.23 KB | batigolix |
help_openid_after.png | 109.02 KB | arianek |
Comments
Comment #1
amye CreditAttribution: amye commentedGenerally, I like it.
No changes from me.
Mistakenly marked 'active' instead of 'needs review'.
Comment #2
amye CreditAttribution: amye commentedChanging status to 'needs' review'.
Comment #3
arianek CreditAttribution: arianek commentedComment #4
arianek CreditAttribution: arianek commentedComment #5
batigolixi would mention a couple of well known openid providers (yahoo, flickr, google?)
punctuation:
"eg. myusername.openidprovider.com"
=>
"e.g., myusername.openidprovider.com"
instead of
"At no point does your site record, or need to record the user's OpenID password."
i would write
"At no point does your site record the user's OpenID password."
this help text seems more detailed than other help texts we've writing before. but i like it. maybe we should write them all like this one (for drupal 9 :)
Comment #6
batigolixforgot the status
Comment #7
batigolixthe module name stays capitalized as the official name of OpenID is OpenID (with capital O)
Comment #8
batigoliximproved patch
Comment #9
arianek CreditAttribution: arianek commentedremoved the mentions of microsoft, etc. the openid site has info on that stuff. cleaned up the language a bit more, and a couple more code tweaks.
Comment #10
jhodgdonLooks good.
Comment #11
webchickCool, committed to HEAD!
This one contains a bit more "how to" in the "Uses" section than other pages I've seen. I don't think this is necessarily a bad thing, particularly with modules that are fairly technical, like this one. We might also want to employ a similar strategy over at #640216: Help File Fixup: Update status module.