On Nov 29, 2005, at 4:45 PM, Phillip J. Eby wrote:

At 03:53 PM 11/29/2005 -0800, Ted Leung wrote:
Building Chandler Parcels <http://svn.osafoundation.org/chandler/ branches/Chandler_0.6/chandler/distrib/docs/ BuildingChandlerParcels.html> is a revision of this year's PyCon paper <http://wiki.osafoundation.org/bin/view/Documentation/ BuildingChandlerParcels> to reflect all the changes that we've made
in 0.6.   The goal of the document is not to be a detail tutorial on
how to write a parcel, but to present enough detail so that someone
has a general idea of the process.

I'd love it if a few people could take some time to review it and
suggest improvements.

* PeriodicTask automatically creates a new view, so you don't need the 'createView()' call. 'self.view = item.itsView' is sufficient.

Ok, I didn't know that


* It would probably be easier to see the schema as a whole if the attribute definitions were one-liners, given that the definitions are all quite short. Right now the FeedChannel class can't be read in one screenful.

I can read it in a screenful ;-). I'll fix this, probably after incorporating some of Alec's ideas to shorten.


* It seems like all the init code should be wrapped in a def installParcel() block; right now that's just alluded to in the text, and it makes the question of where 'parcel' comes from, and when all the code is executed, a bit confusing.

This is a good idea. I wasn't really happy with that section (it was sort of vestigial from the initial version), and I think this will really help


* The link at the bottom that says "There is a more detailed tutorial on the workings of the Feeds parcel that will explain more of the details of writing a parcel." is actually a link to the same page. :)

That's because I'm still not quite sure where that link should point..

Thanks for the feedback,

Ted
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

Open Source Applications Foundation "Dev" mailing list
http://lists.osafoundation.org/mailman/listinfo/dev

Reply via email to