-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for Python 3 #62
Comments
@agitator What's up with the plone5 and the plone5degrok branches? Were they just not finished, have there been any problems? |
plone5degrok was an intermediate branch and can be deleted. |
I started working on this in https://github.com/collective/collective.contact.core/tree/plone5-only |
Removed all grok remains in plone5-only ... c.contact.widget would be next to degrok but as I got a better in depth view of the whole addon, I'd rather switch to z3c.relationfield/RelationChoice/RelationList with pat-relateditems than to upgrade and maintain the existing widget package. UI and functionalitywise I don't see anything missing... and will try it out in the coming week. An upgrade step would be necessary to migrate existing field values which would be also on my list. I think that would be the most future proof way to do it. |
Looked into using z3c.relationfield but gave up when I saw the to me weird seeming interface dependencies between core and widget, especially IContactContent... still think it would make sense. Meanwhile started further upgrading/degroking the widget in https://github.com/collective/collective.contact.widget/blob/plone5-mockup-degrok |
https://github.com/collective/collective.contact.core/tree/plone5-only seems to work on py3 now robot is still haunting me an I'll probably give up on that - python test are green but will continue with manual testing |
@sgeulette @gbastien any update on your plans towards Plone5/6 and a 2.0 version? |
Hi @agitator , we plan to start working on Plone5 migration at the end of the year. |
I'm working on that on PLOG.
The text was updated successfully, but these errors were encountered: