launchpad-reviewers team mailing list archive

Thread
Date

Re: [Merge] lp:~cjwatson/launchpad/zope.tales-upgrade into lp:launchpad

To: Colin Watson <cjwatson@xxxxxxxxxxxxx>
From: Celso Providelo <celso.providelo@xxxxxxxxxxxxx>
Date: Tue, 20 Jan 2015 18:35:55 -0000
In-reply-to: <20150120153426.13937.21161.launchpad@ackee.canonical.com>
Reply-to: mp+247030@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

I had to check the docs [1], from that I take that it should be required=False, but I am not sure if we use schemas for object assignment in zstorm. If we don't, it would only matter for creation views (IIRC) which is not the case for download counts. 

[1] http://docs.zope.org/zope.schema/narr.html#data-modeling-concepts
-- 
https://code.launchpad.net/~cjwatson/launchpad/zope.tales-upgrade/+merge/247030
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.

References

[Merge] lp:~cjwatson/launchpad/zope.tales-upgrade into lp:launchpad
From: Colin Watson, 2015-01-20