launchpad-reviewers team mailing list archive

[Jürgen Gmach <mp+420671@xxxxxxxxxxxxxxxxxx>]

Jürgen Gmach has proposed merging ~jugmac00/launchpad:fix-docstring-for-visible-email-addresses into launchpad:master.

Commit message:
Fix docstring for Personview.visible_email_addresses

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~jugmac00/launchpad/+git/launchpad/+merge/420671
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~jugmac00/launchpad:fix-docstring-for-visible-email-addresses into launchpad:master.

diff --git a/lib/lp/registry/browser/person.py b/lib/lp/registry/browser/person.py
index 99526b7..7005a54 100644
--- a/lib/lp/registry/browser/person.py
+++ b/lib/lp/registry/browser/person.py
@@ -1859,10 +1859,11 @@ class PersonView(LaunchpadView, FeedsMixin, ContactViaWebLinksMixin):
         """The list of email address that can be shown.
 
         The list contains email addresses when the EmailAddressVisibleState's
-        PUBLIC or ALLOWED attributes are True. The preferred email
-        address is the first in the list, the other validated email addresses
-        are not ordered. When the team is the context, only the preferred
-        email address is in the list.
+        PUBLIC or ALLOWED attributes are True.
+        The preferred email address is the first in the list, the other
+        validated email addresses are ordered alphabetically.
+        When the team is the context, only the preferred email address is in
+        the list.
 
         :return: A list of email address strings that can be seen.
         """