dtolabs / yana2

Yet Another Node Authority ... because you wanted an agile Real-Time Service Model (RTSM)
8 stars 1 forks source link

Remove status property from Node #175

Closed ahonor closed 12 years ago

ahonor commented 12 years ago

Remove the status property from the Node domain class. An attribute can be defined for users interested in a property like status.

See #169

orubel commented 12 years ago

Well this was for 'enabled','disabled','pending',etc. This allows people to build nodes without putting them into service as well as providing quick search without nested queries and/or lots of joins . Highly recommend not moving since this can be future function if not ready to enable yet but should be at this level in the tree.

It would be impossible by the way to build a node without putting it into service without building said attribute; with status IN node, you have said ability. You would have to build out the node to provide this capability first in the model you are describing. By default, this exists in current model.

ahonor commented 12 years ago

Attributes should be easy to search as that is a key use pattern. We can optimize that for ease of use.

orubel commented 12 years ago

You would still need to be able to create the node and have it in a holding pattern without having to create any attributes, IMHO. Otherwise you are stating to users that this attribute has to be created EVERY time which is an annoyance.