insoshi

Comments and changes to this ticket

• 

  Jorge Dias June 6th, 2008 @ 07:25 PM

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

  Hi Michael, I made a fix for this but haven't commited it yet. Wanted to check out first if you like the approach

  I modified the person model to look like this

  has_many :activities, :through => :feeds, :order => 'activities.created_at DESC',

  :limit => FEED_SIZE,

  :conditions => ["people.deactivated = ?", false],

  :include => :person

  I made the include because I didn't want write sql. =)

  If you like it I can commit it an make the tests.

  Let me know what you think...

• 

  Michael Hartl June 7th, 2008 @ 05:13 AM

  I like it. Let's see what the tests look like.

• 

  Jorge Dias June 9th, 2008 @ 04:01 PM

  Fixed this in my fork! Check out the tests.

• 

  Michael Hartl June 17th, 2008 @ 04:39 PM

  • → Milestone changed from “” to “Miscellaneous polish”
• 

  Michael Hartl September 15th, 2008 @ 04:18 PM

  • → State changed from “new” to “resolved”
  • → Tag cleared.