suggestions?


On Wed, Dec 4, 2013 at 11:25 PM, Tyler Wymer <notifications@github.com>wrote:

> In corehq/apps/commtrack/models.py:
>
> >      """
> >      wrapper/helper for transactions
> >      """
> > -
> > -    value = IntegerProperty()
> > +    domain = StringProperty()
> > +    timestamp = DateTimeProperty()
> > +    location_id = StringProperty() # location record id
> > +    case_id = StringProperty() # supply point case id
>
> If property names need comments to explain what they are, shouldn't they
> be named differently?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/dimagi/commcare-hq/pull/1995/files#r8110723>
> .
>