insoshi

Comments and changes to this ticket

• 

  Michael Hartl April 9th, 2008 @ 12:18 PM

  • → State changed from “new” to “open”
• 

  Jorge Dias April 23rd, 2008 @ 06:11 AM

  • → Assigned user changed from “” to “Jorge Dias”
• 

  Jorge Dias April 28th, 2008 @ 06:54 AM

  Fixed this in my fork, added two callbacks to the person model.

  When rails 2.1 comes out, or if we move to edge it should be easier to do with one callback only

• 

  Michael Hartl April 28th, 2008 @ 10:21 AM

  I've realized that we probably don't want this quite yet. The reason is that many people will edit their profiles several times in succession. These should not appear as separate events, though, since each change is likely to be minor. What we really want is to aggregate the activities so that all changes within, say, 15 minutes get rolled into one activity item.

  I'd prefer to defer this till later---unless, of course, you want to implement the aggregation.

• 

  Jorge Dias April 30th, 2008 @ 08:29 AM

  What do you mean by implementing the aggregation?

  Would it be something as simple as checking if you have a previous activity, generated within a time frame on the same item?

  Or do you mean a more complex solution? Any ideas or suggestions so I could work on this?

• 

  Evan Dorn May 5th, 2008 @ 05:37 PM

  how about like (pseudocode, I haven't actually looked at the code):

  
  
  user has just edited profile
  
  
  if (Time.now - last_update_time(user) < 1.hour)
  
  delete_old_profile_update_event
  
  end
  
  create_profile_update_event
  
  

  Wouldn't that be sufficient?

• 

  Michael Hartl May 7th, 2008 @ 12:16 PM

  • → State changed from “open” to “resolved”