[buddypress-trac] [BuddyPress Trac] #4636: Reinstate activity item for user profile changes

buddypress-trac noreply at wordpress.org
Sun Feb 9 20:21:34 UTC 2014


#4636: Reinstate activity item for user profile changes
--------------------------+------------------
 Reporter:  DJPaul        |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  2.0
Component:  XProfile      |     Version:  1.1
 Severity:  normal        |  Resolution:
 Keywords:  has-patch     |
--------------------------+------------------
Changes (by boonebgorges):

 * keywords:  needs-patch => has-patch


Comment:

 I've done some thinking about ericlewis's original patch, and come to a
 couple conclusions.

 - I may be going a bit overboard, but it does seem that changing return
 value format of existing functions is going too far, just to do what we're
 doing here.
 - We should have a throttle, so that you don't get more than one update
 for a certain period of time (I chose 2 hours)
 - Field-by-field updates are not helpful, and the extra mechanisms needed
 for lumping them into a single update ("Boone updated Hometown, Display
 Name...") would add a lot of overhead for not much gain.
 - I think we'd annoy a lot of people with the "Eric updated their profile"
 syntax.

 4636.02.patch addresses these concerns. I went with "Eric's profile was
 updated", which is the best way I could think of in English to say it
 without resorting to user gender. Updates are throttled in two-hour
 chunks.

 Feedback welcome.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/4636#comment:11>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac


More information about the buddypress-trac mailing list