#10902: New collections (merged plip)
---------------------+------------------------------------------------------
 Reporter:  elvix    |       Owner:  elvix
     Type:  PLIP     |      Status:  new  
 Priority:  minor    |   Milestone:  4.1  
Component:  Unknown  |    Keywords:       
---------------------+------------------------------------------------------

Comment(by rossp):

 Reviewed: https://svn.plone.org/svn/plone/buildouts/plone-
 coredev/branches/4.1/plips/plip10902-review-rossp.txt


 The implementation is slick and really nice to use.  It's a vast
 improvement over what we have currently.  I'd love to see it included
 in 4.1.

 Having said that I don't think I'm comfortable merging it as-is, but
 the changes required seem to be pretty doable.  If more complete
 docstrings, comments, and interfaces are added, the test failures are
 addressed, and the BadRequest errors are addressed, I'm a strong +1
 for merging.

-- 
Ticket URL: <http://dev.plone.org/plone/ticket/10902#comment:10>
Plone <http://plone.org>
Plone Content Management System
_______________________________________________
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories

Reply via email to