Navigation Menu

Skip to content
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

Plone5 #35

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Plone5 #35

wants to merge 11 commits into from

Conversation

pbauer
Copy link
Member

@pbauer pbauer commented Apr 15, 2016

No description provided.

@gborelli
Copy link
Contributor

In the packages that need two different versions to maintain the compatibility with plone4 I've already merged the plone5 branch and used a specific branch for plone4.
Probably we don't need to pin plone5 branch there.

I've also started to work with Docker for developing, if you are interested. See.:

@pbauer
Copy link
Member Author

pbauer commented Apr 15, 2016

Great. I'm working on the robot-tests for now.

@pbauer
Copy link
Member Author

pbauer commented Apr 15, 2016

@gborelli I cannot get the robot-tests of the bundle to work properly. Even with a complely new test-setup (based on bobtemplates.plone) the traverser ++add++ does not work on the site created for the tests so I cannot create any content in robot-tests unless I use python. I real-life it works and seems to be a problem with the dependecies.

@pbauer
Copy link
Member Author

pbauer commented Apr 15, 2016

Also: should we aim to make a new release of collective.geo.bundle that works with plone5 and also pulls in collective.geo.behaviour?

@pbauer
Copy link
Member Author

pbauer commented Apr 22, 2016

@gborelli I reworked the test-setup and got a little further with finding the culprit. I added the failing test collective.geo.bundle.tests.test_setup.TestSetup.test_create_document that shows that instead of the add-view for documents a instance of collective.geo.mapwidget.browser.widget.MapWidgets is returned. I have no real idea what that is actually good for and why it is registered the way it is. Can you please have a look at that?

@pbauer
Copy link
Member Author

pbauer commented Apr 22, 2016

@pbauer
Copy link
Member Author

pbauer commented Jun 23, 2016

@gborelli ping

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants