<br><div class="gmail_quote">On Tue, Jan 3, 2012 at 16:35, Héctor Velarde <span dir="ltr"><<a href="mailto:hector.velarde@gmail.com">hector.velarde@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On 03/01/12 11:42, Rok Garbas - <a href="http://gmail.com" target="_blank">gmail.com</a> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
which functionalities are not needed?<br>
</blockquote>
<br></div>
all the migrating comments stuff and so on; this should be outside this package to keep it focused and clean.<br>
<br>
on the SSO support, I not sure we need to implement so many options on it... things like login popup width and height are just too much... but that's only my personal opinion ;-)<br>
<br>
also, I noticed that nobody has merged the changes on saibatizoku's branch to add Dexterity compatibility:<br>
<br>
<a href="http://dev.plone.org/browser/collective/collective.disqus/branches/dexterity-uuid" target="_blank">http://dev.plone.org/browser/<u></u>collective/collective.disqus/<u></u>branches/dexterity-uuid</a><br>
<br>
I think it would be a good idea to do the following:<br>
<br>
1. move the code to <a href="https://github.com/collective" target="_blank">https://github.com/collective</a><br>
2. remove compatibility with versions prior Plone 4.1<br>
3. replace plone.app.controlpanel with plone.app.registry<br>
4. improve test coverage<br></blockquote><div><br></div><div><br></div><div>I think we have a plan. </div><div><br></div><div><br></div><div>best,</div><div>ea</div></div>