Would like to make various changes to the README.txt. For one, rewrite "Security note" section to improve clarity. Sentences like this seem confusing:

Help texts on how can you memorize such things (like shifting a word one row up the keyboard and so on).
CommentFileSizeAuthor
#11 password_policy-improve_readme-2455881-11.patch6.19 KBaohrvetpv
#10 password_policy-improve_readme-2455881-10.patch6.31 KBaohrvetpv
#7 password_policy-improve_readme-2455881-7.patch5.6 KBaohrvetpv
#2 password_policy-improve_readme-2455881-2.patch4.09 KBaohrvetpv
#1 password_policy-improve_readme-2455881-1.patch4.09 KBaohrvetpv

Comments

aohrvetpv’s picture

aohrvetpv commented
StatusFileSize
new password_policy-improve_readme-2455881-1.patch4.09 KB

First draft

Summary of main changes:
- Rephrase introductory sentence (see #2455909: First sentence of project page is perhaps misleading for rationale).
- Rewrite security note section.
- Remove references to specific password managers. I think 1Password specifically should not be recommended since it is not free and open source software. SuperGenPass is probably OK, but KeePassX is probably more popular. Rather than picking favorites, link to Wikipedia list.
- Add to list of sponsoring organizations per project page.
- Rewrap some lines.

Please review!

aohrvetpv’s picture

aohrvetpv commented
StatusFileSize
new password_policy-improve_readme-2455881-2.patch4.09 KB

Remove a trailing space.

aohrvetpv’s picture

aohrvetpv commented
Status: Active » Needs review

The last submitted patch, 1: password_policy-improve_readme-2455881-1.patch, failed testing.

Status: Needs review » Needs work

The last submitted patch, 2: password_policy-improve_readme-2455881-2.patch, failed testing.

Status: Needs work » Needs review

AohRveTPV queued 2: password_policy-improve_readme-2455881-2.patch for re-testing.

aohrvetpv’s picture

aohrvetpv commented
StatusFileSize
new password_policy-improve_readme-2455881-7.patch5.6 KB

Make README.txt formatting more consistent with other README.txt files, by selectively following some of the suggestions from https://www.drupal.org/node/2181737.

AohRveTPV queued 7: password_policy-improve_readme-2455881-7.patch for re-testing.

Status: Needs review » Needs work

The last submitted patch, 7: password_policy-improve_readme-2455881-7.patch, failed testing.

aohrvetpv’s picture

aohrvetpv commented
Status: Needs work » Needs review
StatusFileSize
new password_policy-improve_readme-2455881-10.patch6.31 KB

- Update to apply on latest README.txt.
- Combine "Security Considerations" and "Limitations" sections into single "Limitations" section.
- Shorten wording of a sentence.

aohrvetpv’s picture

aohrvetpv commented
StatusFileSize
new password_policy-improve_readme-2455881-11.patch6.19 KB

- Fix bullet error.
- Remove redundant word "constraint" from each of the listed constraints.

aohrvetpv’s picture

aohrvetpv commented

  • AohRveTPV committed cb7c52c on 7.x-1.x 
    Issue #2455881 by AohRveTPV: Improve README.txt
aohrvetpv’s picture

aohrvetpv commented
Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Needs review » Patch (to be ported)
aohrvetpv’s picture

aohrvetpv commented
Status: Patch (to be ported) » Closed (outdated)

6.x-1.x is no longer supported.