AmpersandJS / ampersand-model

Observable objects, for managing state in applications.
MIT License
84 stars 31 forks source link

A collection declared as a prop introduces errors. #59

Closed EvanCarroll closed 9 years ago

EvanCarroll commented 9 years ago

I think if you have a collection and a prop by the same name, a fatal error should be thrown when one tries to override the other. My code only works as a collection..

module.exports = BaseModel.extend({        
  props: {                                 
    clientid: 'number'                     
    // , events: 'object'                     
  }                                        
  , collections: { events: NE } 
cdaringe commented 9 years ago

@pgilad recently did just this. https://github.com/AmpersandJS/ampersand-state/pull/192. looks like the &-model deps just need to be bumped to ampersand-state@4.6.x+

kamilogorek commented 9 years ago

ampersand-state has been bumped, therefore closing this one.