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. -- Mailing list: https://launchpad.net/~maas-devel Post to : [email protected] Unsubscribe : https://launchpad.net/~maas-devel More help : https://help.launchpad.net/ListHelp
