Date: Mon Sep 25 22:41:02 2006
New Revision: 11003
review/plip-168-iterate/COMMENTS.txt (contents, props changed)
--- (empty file)
+++ review/plip-168-iterate/COMMENTS.txt Mon Sep 25 22:41:02 2006
@@ -0,0 +1,43 @@
+Please put any comments related to reviewing this bundle into this file.
+Review by Helge Tesdal
+PLIP168 Iterate provides in-place staging in Plone for improved editing of
+2. Test setup
+This was tested with Zope 2.10 from SVN and the review bundle.
+svn co https://svn.plone.org/svn/plone/review/plip-168-iterate Products
+3. How does it work
+The action menu has a 'check out' option, to create a copy of the published
+content. On checkout, an event is fired, and a reference linking the copy to
+the original is added. The copy has 'check in' and 'cancel checkout' actions.
+When checking in, an event is fired, and the published content is replaced by
+the edited copy.
+4. State of code
+Great. Cleanly laid out, customizable by adapters and events. Very confidence
+5. Suggested improvements
+When I tried to view the locked front page I had made a copy of, I got an
+error and was unable to view the page until I checked in the copy. Didn't
+look into it - assuming it's a minor issue, but has to be fixed before merge.
+There are a couple of items in the todo list that will hopefully get some more
+attention if iterate is included in Plone 3, but iterate is useful in Plone 3
+even without those items.
++1 to including this in Plone 3.