maas-devel team mailing list archive

[Gavin Panella <gavin.panella@xxxxxxxxxxxxx>]

Review: Approve

Okay, looks good.


[1]

> It does something somewhat right now: it stores the date with the
> tz.

Yes, but which timezone? It's not specified anywhere.


[3]

>> Does Django provide a decent repr around this? In other words, it
>> would be nice to have a decent repr :)
>
> The default __repr__ is <Node Ox1234>.

Okay, not very exciting. I guess we could introduce our own magical
base class at some point.


[7]

> That's the way it is, the validation happens when you call
> full_clean on the model (see
> https://docs.djangoproject.com/en/dev/ref/models/instances/#validating-objects).

Cool, thanks for the link.

-- 
https://code.launchpad.net/~rvb/maas/maas-basic-model/+merge/88890
Your team MaaS Developers is subscribed to branch lp:maas.