search

plone / plone.app.ldap

Plone UI and integration of the functionality provided by LDAPMultiPlugins, LDAPUserFolder and PloneLDAP
3 stars 10 forks source link

Add uninstall code #37

Closed mauritsvanrees closed 5 years ago

mauritsvanrees commented 5 years ago

Fixes issues #14 and #26. The branch for this PR builds on the branch for PR #36 to fix Travis: it contains all its commits.

mister-roboto commented 5 years ago

@mauritsvanrees thanks for creating this Pull Request and help improve Plone!

To ensure that these changes do not break other parts of Plone, the Plone test suite matrix needs to pass.

Whenever you feel that the pull request is ready to be tested, either start all jenkins jobs pull requests by yourself, or simply add a comment in this pull request stating:

@jenkins-plone-org please run jobs

With this simple comment all the jobs will be started automatically.

Happy hacking!

mauritsvanrees commented 5 years ago

No Jenkins job needed, as this is not core Plone.

mauritsvanrees commented 5 years ago

Travis passes. Ready for review.