search

richardkmichael / buy-my-beer

Make broken builds a thing of the past! Broken builds cost you a beer!
1 stars 0 forks source link

Project.users should be renamed to Project.collaborators. #27

Open richardkmichael opened 13 years ago

richardkmichael commented 13 years ago 
validates :users, :presence => true
has_many :collaborations, :dependent => :destroy
has_many :users, :through => :collaborations

Would be nicer as:

validates :collaborators, :presence => true
has_many :collaborations, :dependent => :destroy
has_many :collaborators, :through => :collaborations, :class => User

I think.

franckverrot commented 13 years ago

agreed

On Mon, Sep 5, 2011 at 9:28 PM, richardkmichael < reply@reply.github.com>wrote:

validates :users, :presence => true
has_many :collaborations, :dependent => :destroy
has_many :users, :through => :collaborations

Would be nicer as:

validates :collaborators, :presence => true
has_many :collaborations, :dependent => :destroy
has_many :collaborators, :through => :collaborations, :class => User

I think.

Reply to this email directly or view it on GitHub: https://github.com/richardkmichael/buy-my-beer/issues/27

Franck Verrot https://github.com/cesario https://twitter.com/cesariogw